reverting 11d12bd6e1dcd9610fa26e97d25e7ad553e8ffa5

passing local everywhere violate the layering

mercurial/changelog.py

@@ -11,9 +11,8 b' from demandload import demandload'
 demandload(globals(), "os time util")
 
 class changelog(revlog):
-    def __init__(self, opener, local=True):
-        revlog.__init__(self, opener, "00changelog.i", "00changelog.d",
-                        local=local)
+    def __init__(self, opener):
+        revlog.__init__(self, opener, "00changelog.i", "00changelog.d")
 
     def extract(self, text):
         if not text:

mercurial/filelog.py

@@ -11,11 +11,10 b' from demandload import *'
 demandload(globals(), "bdiff")
 
 class filelog(revlog):
-    def __init__(self, opener, path, local=True):
+    def __init__(self, opener, path):
         revlog.__init__(self, opener,
                         os.path.join("data", self.encodedir(path + ".i")),
-                        os.path.join("data", self.encodedir(path + ".d")),
-                        local=local)
+                        os.path.join("data", self.encodedir(path + ".d")))
 
     # This avoids a collision between a file named foo and a dir named
     # foo.i or foo.d

mercurial/localrepo.py

@@ -30,8 +30,8 b' class localrepository(object):'
         self.ui = ui
         self.opener = util.opener(self.path)
         self.wopener = util.opener(self.root)
-        self.manifest = manifest.manifest(self.opener, local=self.local())
-        self.changelog = changelog.changelog(self.opener, local=self.local())
+        self.manifest = manifest.manifest(self.opener)
+        self.changelog = changelog.changelog(self.opener)
         self.tagscache = None
         self.nodetagscache = None
         self.encodepats = None
@@ -161,7 +161,7 b' class localrepository(object):'
 
     def file(self, f):
         if f[0] == '/': f = f[1:]
-        return filelog.filelog(self.opener, f, local=self.local())
+        return filelog.filelog(self.opener, f)
 
     def getcwd(self):
         return self.dirstate.getcwd()

mercurial/manifest.py

@@ -12,11 +12,10 b' from demandload import *'
 demandload(globals(), "bisect array")
 
 class manifest(revlog):
-    def __init__(self, opener, local=True):
+    def __init__(self, opener):
         self.mapcache = None
         self.listcache = None
-        revlog.__init__(self, opener, "00manifest.i", "00manifest.d",
-                        local=local)
+        revlog.__init__(self, opener, "00manifest.i", "00manifest.d")
 
     def read(self, node):
         if node == nullid: return {} # don't upset local cache

mercurial/revlog.py

@@ -177,7 +177,7 b' class revlog(object):'
     remove data, and can use some simple techniques to avoid the need
     for locking while reading.
     """
-    def __init__(self, opener, indexfile, datafile, local=True):
+    def __init__(self, opener, indexfile, datafile):
         """
         create a revlog object
 
@@ -188,7 +188,6 b' class revlog(object):'
         self.datafile = datafile
         self.opener = opener
         self.cache = None
-        self.local = local # XXX only needed because statichttp
 
         try:
             i = self.opener(self.indexfile).read()
@@ -661,6 +660,7 b' class revlog(object):'
         changesets. parent is parent[0]
         """
         revs = [self.rev(n) for n in nodelist]
+        needed = dict.fromkeys(revs, 1)
 
         # if we don't have any revisions touched by these changesets, bail
         if not revs:
@@ -671,16 +671,6 b' class revlog(object):'
         p = self.parents(self.node(revs[0]))[0]
         revs.insert(0, self.rev(p))
 
-        if self.local:
-            mm = self.opener(self.datafile)
-            def chunk(r):
-                o = self.start(r)
-                l = self.length(r)
-                mm.seek(o)
-                return decompress(mm.read(l))
-        else:
-            # XXX: statichttp workaround
-            needed = dict.fromkeys(revs[1:], 1)
             # for each delta that isn't contiguous in the log, we need to
             # reconstruct the base, reconstruct the result, and then
             # calculate the delta. We also need to do this where we've
@@ -726,15 +716,14 b' class revlog(object):'
                 for r, l in span[2]:
                     chunks[r] = decompress(data[pos: pos + l])
                     pos += l
-            def chunk(r):
-                return chunks[r]
 
         # helper to reconstruct intermediate versions
         def construct(text, base, rev):
-            bins = [chunk(r) for r in xrange(base + 1, rev + 1)]
+            bins = [chunks[r] for r in xrange(base + 1, rev + 1)]
             return mdiff.patches(text, bins)
 
         # build deltas
+        deltas = []
         for d in xrange(0, len(revs) - 1):
             a, b = revs[d], revs[d + 1]
             n = self.node(b)
@@ -746,7 +735,7 b' class revlog(object):'
             if a + 1 != b or self.base(b) == b:
                 if a >= 0:
                     base = self.base(a)
-                    ta = chunk(self.base(a))
+                    ta = chunks[self.base(a)]
                     ta = construct(ta, base, a)
                 else:
                     ta = ""
@@ -756,11 +745,11 b' class revlog(object):'
                     base = a
                     tb = ta
                 else:
-                    tb = chunk(self.base(b))
+                    tb = chunks[self.base(b)]
                 tb = construct(tb, base, b)
                 d = self.diff(ta, tb)
             else:
-                d = chunk(b)
+                d = chunks[b]
 
             p = self.parents(n)
             meta = n + p[0] + p[1] + lookup(n)

mercurial/statichttprepo.py

@@ -31,12 +31,10 b' class statichttprepository(localrepo.loc'
         self.path = (path + "/.hg")
         self.ui = ui
         self.opener = opener(self.path)
-        self.manifest = manifest.manifest(self.opener, local=self.local())
-        self.changelog = changelog.changelog(self.opener, local=self.local())
+        self.manifest = manifest.manifest(self.opener)
+        self.changelog = changelog.changelog(self.opener)
         self.tagscache = None
         self.nodetagscache = None
-        self.encodepats = None
-        self.decodepats = None
 
     def dev(self):
         return -1