[PATCH 1 of 6 V3] scmutil: rename classes from "opener" to "vfs"

FUJIWARA Katsunori foozy at lares.dti.ne.jp
Thu Aug 30 13:14:17 CDT 2012


# HG changeset patch
# User FUJIWARA Katsunori <foozy at lares.dti.ne.jp>
# Date 1346346389 -32400
# Node ID 60ea1d2867392542835136df48517a3038eb96fe
# Parent  fc14953e8e34667181cd0d492825342b5e1e880b
scmutil: rename classes from "opener" to "vfs"

For backwards compatibility, aliases for the old names are added,
except for "abstractopener", "statichttpopener" and "_fncacheopener",
because these are not used in Mercurial core implementation after this
patch.

"_fncacheopener" was only referred in "fncachestore" constructor, so
this patch also renames from "_fncacheopener" to "_fncachevfs" there.

diff -r fc14953e8e34 -r 60ea1d286739 mercurial/scmutil.py
--- a/mercurial/scmutil.py	Tue Aug 28 17:59:08 2012 -0500
+++ b/mercurial/scmutil.py	Fri Aug 31 02:06:29 2012 +0900
@@ -167,7 +167,7 @@
         # want to add "foo/bar/baz" before checking if there's a "foo/.hg"
         self.auditeddir.update(prefixes)
 
-class abstractopener(object):
+class abstractvfs(object):
     """Abstract base class; cannot be instantiated"""
 
     def __init__(self, *args, **kwargs):
@@ -219,8 +219,8 @@
     def makedirs(self, path=None, mode=None):
         return util.makedirs(self.join(path), mode)
 
-class opener(abstractopener):
-    '''Open files relative to a base directory
+class vfs(abstractvfs):
+    '''Operate files relative to a base directory
 
     This class is used to hide the details of COW semantics and
     remote file access from higher level code.
@@ -327,8 +327,10 @@
         else:
             return self.base
 
-class filteropener(abstractopener):
-    '''Wrapper opener for filtering filenames with a function.'''
+opener = vfs
+
+class filtervfs(abstractvfs):
+    '''Wrapper vfs for filtering filenames with a function.'''
 
     def __init__(self, opener, filter):
         self._filter = filter
@@ -337,6 +339,8 @@
     def __call__(self, path, *args, **kwargs):
         return self._orig(self._filter(path), *args, **kwargs)
 
+filteropener = filtervfs
+
 def canonpath(root, cwd, myname, auditor=None):
     '''return the canonical path of myname, given cwd and root'''
     if util.endswithsep(root):
diff -r fc14953e8e34 -r 60ea1d286739 mercurial/statichttprepo.py
--- a/mercurial/statichttprepo.py	Tue Aug 28 17:59:08 2012 -0500
+++ b/mercurial/statichttprepo.py	Fri Aug 31 02:06:29 2012 +0900
@@ -64,7 +64,7 @@
     urlopener = url.opener(ui, authinfo)
     urlopener.add_handler(byterange.HTTPRangeHandler())
 
-    class statichttpopener(scmutil.abstractopener):
+    class statichttpvfs(scmutil.abstractvfs):
         def __init__(self, base):
             self.base = base
 
@@ -74,7 +74,7 @@
             f = "/".join((self.base, urllib.quote(path)))
             return httprangereader(f, urlopener)
 
-    return statichttpopener
+    return statichttpvfs
 
 class statichttppeer(localrepo.localpeer):
     def local(self):
diff -r fc14953e8e34 -r 60ea1d286739 mercurial/store.py
--- a/mercurial/store.py	Tue Aug 28 17:59:08 2012 -0500
+++ b/mercurial/store.py	Fri Aug 31 02:06:29 2012 +0900
@@ -367,7 +367,7 @@
             self._load()
         return iter(self.entries)
 
-class _fncacheopener(scmutil.abstractopener):
+class _fncachevfs(scmutil.abstractvfs):
     def __init__(self, op, fnc, encode):
         self.opener = op
         self.fncache = fnc
@@ -387,7 +387,7 @@
         op.createmode = self.createmode
         fnc = fncache(op)
         self.fncache = fnc
-        self.opener = _fncacheopener(op, fnc, encode)
+        self.opener = _fncachevfs(op, fnc, encode)
 
     def join(self, f):
         return self.path + '/' + self.encode(f)


More information about the Mercurial-devel mailing list