Changeset d71adaf in trunk

Timestamp:

2009-07-04T03:43:01Z (16 years ago)

Author:

kevan <kevan@…>

Branches:

master

Children:

5501983

Parents:

34213cd

Message:

Use CachingDict? instead of dict in dirnode.py

File:

• src/allmydata/dirnode.py (modified) (3 diffs)

src/allmydata/dirnode.py ¶

@@ -171 +171 @@
         # URI to create our own.
         self._node = self.filenode_class(self._client)
-        empty_contents = self._pack_contents({})
+        empty_contents = self._pack_contents(CachingDict())
         d = self._node.create(empty_contents, keypair_generator, keysize=keysize)
         d.addCallback(self._filenode_created)
@@ -225 +225 @@
         # an empty directory is serialized as an empty string
         if data == "":
-            return {}
+            return CachingDict()
         writeable = not self.is_readonly()
-        children = {}
+        children = CachingDict()
         position = 0
         while position < len(data):
@@ -244 +244 @@
             metadata = simplejson.loads(metadata_s)
             assert isinstance(metadata, dict)
-            children[name] = (child, metadata)
+            children.set_both_items(name, entry, (child, metadata))
         return children
 
     def _pack_contents(self, children):
         # expects children in the same format as _unpack_contents
-        assert isinstance(children, dict)
+        assert isinstance(children, CachingDict)
         entries = []
         for name in sorted(children.keys()):
-            child, metadata = children[name]
-            assert isinstance(name, unicode)
-            assert IFilesystemNode.providedBy(child), (name,child)
-            assert isinstance(metadata, dict)
-            rwcap = child.get_uri() # might be RO if the child is not writeable
-            if rwcap is None:
-                rwcap = ""
-            assert isinstance(rwcap, str), rwcap
-            rocap = child.get_readonly_uri()
-            if rocap is None:
-                rocap = ""
-            assert isinstance(rocap, str), rocap
-            entry = "".join([netstring(name.encode("utf-8")),
+            entry, metadata = children.get_both_items(name)
+            if entry == None:
+                child, metadata = metadata
+                assert isinstance(name, unicode)
+                assert IFilesystemNode.providedBy(child), (name,child)
+                assert isinstance(metadata, dict)
+                rwcap = child.get_uri() # might be RO if the child is not writeable
+                if rwcap is None:
+                    rwcap = ""
+                assert isinstance(rwcap, str), rwcap
+                rocap = child.get_readonly_uri()
+                if rocap is None:
+                    rocap = ""
+                assert isinstance(rocap, str), rocap
+                entry = "".join([netstring(name.encode("utf-8")),
                              netstring(rocap),
                              netstring(self._encrypt_rwcap(rwcap)),