changeset 4605:9da7a4da650f

dirstate: lazify initdirs
author Matt Mackall <mpm@selenic.com>
date Mon, 18 Jun 2007 13:24:33 -0500
parents c867c11426ad
children 2651099c5018
files mercurial/dirstate.py
diffstat 1 files changed, 9 insertions(+), 15 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/dirstate.py	Mon Jun 18 13:24:33 2007 -0500
+++ b/mercurial/dirstate.py	Mon Jun 18 13:24:33 2007 -0500
@@ -20,7 +20,6 @@
         self.root = root
         self.dirty = 0
         self.ui = ui
-        self.dirs = None
         self.ignorefunc = None
         self._slash = None
 
@@ -47,6 +46,11 @@
             except IOError, err:
                 if err.errno != errno.ENOENT: raise
             return self.pl
+        elif name == 'dirs':
+            self.dirs = {}
+            for f in self.map:
+                self.updatedirs(f, 1)
+            return self.dirs
         else:
             raise AttributeError, name
 
@@ -256,24 +260,16 @@
     def copies(self):
         return self.copymap
 
-    def initdirs(self):
-        if self.dirs is None:
-            self.dirs = {}
-            for f in self.map:
-                self.updatedirs(f, 1)
-
     def updatedirs(self, path, delta):
-        if self.dirs is not None:
-            for c in strutil.findall(path, '/'):
-                pc = path[:c]
-                self.dirs.setdefault(pc, 0)
-                self.dirs[pc] += delta
+        for c in strutil.findall(path, '/'):
+            pc = path[:c]
+            self.dirs.setdefault(pc, 0)
+            self.dirs[pc] += delta
 
     def checkinterfering(self, files):
         def prefixes(f):
             for c in strutil.rfindall(f, '/'):
                 yield f[:c]
-        self.initdirs()
         seendirs = {}
         for f in files:
             # shadows
@@ -301,7 +297,6 @@
         if not files: return
         self.markdirty()
         if state == "a":
-            self.initdirs()
             self.checkinterfering(files)
         for f in files:
             if state == "r":
@@ -320,7 +315,6 @@
     def forget(self, files):
         if not files: return
         self.markdirty()
-        self.initdirs()
         for f in files:
             try:
                 del self.map[f]