# HG changeset patch
# User Patrick Mezard <pmezard@gmail.com>
# Date 2011-05-19 20:44:01
# Node ID ce77c275bec3e453d2ebe7d29ae8eb92bf8683d2
# Parent  909ac6b9636b32c42f4d9021530d3127de6b1c89

patch: merge backend setmode() into writelines()

Copy handling will be easier to handle in a single method.

diff --git a/mercurial/patch.py b/mercurial/patch.py
--- a/mercurial/patch.py
+++ b/mercurial/patch.py
@@ -372,9 +372,10 @@ class abstractbackend(object):
         """
         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 @@ class abstractbackend(object):
     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 @@ class fsbackend(abstractbackend):
         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 @@ class fsbackend(abstractbackend):
     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 @@ class workingbackend(fsbackend):
         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 @@ class workingbackend(fsbackend):
         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 @@ class patchfile(object):
                 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 @@ def _applydiff(ui, fp, patcher, backend,
                 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: