patch: merge backend setmode() into writelines()
authorPatrick Mezard <pmezard@gmail.com>
Thu, 19 May 2011 22:44:01 +0200
changeset 14390 ce77c275bec3
parent 14389 909ac6b9636b
child 14391 1e64e1e12195
patch: merge backend setmode() into writelines() Copy handling will be easier to handle in a single method.
mercurial/patch.py
--- a/mercurial/patch.py	Thu May 19 22:44:01 2011 +0200
+++ b/mercurial/patch.py	Thu May 19 22:44:01 2011 +0200
@@ -372,9 +372,10 @@
         """
         raise NotImplementedError
 
-    def writelines(self, fname, lines, mode):
+    def setfile(self, fname, lines, mode):
         """Write lines to target file. mode is a (islink, isexec)
-        tuple, or None if there is no mode information.
+        tuple, or None if there is no mode information. If lines is None,
+        the file must exists and its content is left unchanged.
         """
         raise NotImplementedError
 
@@ -399,10 +400,6 @@
     def exists(self, fname):
         raise NotImplementedError
 
-    def setmode(self, fname, islink, isexec):
-        """Change target file mode."""
-        raise NotImplementedError
-
 class fsbackend(abstractbackend):
     def __init__(self, ui, basedir):
         super(fsbackend, self).__init__(ui)
@@ -420,7 +417,11 @@
         finally:
             fp.close()
 
-    def writelines(self, fname, lines, mode):
+    def setfile(self, fname, lines, mode):
+        if lines is None:
+            if mode:
+                util.setflags(self._join(fname), mode[0], mode[1])
+            return
         if not mode:
             # Preserve mode information
             isexec, islink = False, False
@@ -475,9 +476,6 @@
     def exists(self, fname):
         return os.path.lexists(self._join(fname))
 
-    def setmode(self, fname, islink, isexec):
-        util.setflags(self._join(fname), islink, isexec)
-
 class workingbackend(fsbackend):
     def __init__(self, ui, repo, similarity):
         super(workingbackend, self).__init__(ui, repo.root)
@@ -487,8 +485,8 @@
         self.changed = set()
         self.copied = []
 
-    def writelines(self, fname, lines, mode):
-        super(workingbackend, self).writelines(fname, lines, mode)
+    def setfile(self, fname, lines, mode):
+        super(workingbackend, self).setfile(fname, lines, mode)
         self.changed.add(fname)
 
     def unlink(self, fname):
@@ -501,10 +499,6 @@
         self.copied.append((src, dst))
         self.changed.add(dst)
 
-    def setmode(self, fname, islink, isexec):
-        super(workingbackend, self).setmode(fname, islink, isexec)
-        self.changed.add(fname)
-
     def close(self):
         wctx = self.repo[None]
         addremoved = set(self.changed)
@@ -585,7 +579,7 @@
                 rawlines.append(l)
             lines = rawlines
 
-        self.backend.writelines(fname, lines, mode)
+        self.backend.setfile(fname, lines, mode)
 
     def printfile(self, warn):
         if self.fileprinted:
@@ -1252,11 +1246,12 @@
                 if gp.op == 'RENAME':
                     backend.unlink(pstrip(gp.oldpath))
                 if gp.mode and not first_hunk:
+                    data = None
                     if gp.op == 'ADD':
-                        # Added files without content have no hunk and must be created
-                        backend.writelines(path, [], gp.mode)
-                    else:
-                        backend.setmode(path, gp.mode[0], gp.mode[1])
+                        # Added files without content have no hunk and
+                        # must be created
+                        data = []
+                    backend.setfile(path, data, gp.mode)
             if not first_hunk:
                 continue
             try: