Patchwork [9,of,9] manifestmerge: drop redundant flags calls

login
register
mail settings
Submitter Siddharth Agarwal
Date March 25, 2013, 12:27 a.m.
Message ID <9be8870ad97b8ab042af.1364171273@sid0x220>
Download mbox | patch
Permalink /patch/1185/
State Superseded
Commit b2a36e9b9ccc9b6187efa93970eac7547aeb9632
Headers show

Comments

Siddharth Agarwal - March 25, 2013, 12:27 a.m.
# HG changeset patch
# User Siddharth Agarwal <sid0@fb.com>
# Date 1364169385 25200
#      Sun Mar 24 16:56:25 2013 -0700
# Node ID 9be8870ad97b8ab042af20865c09daf6146c2814
# Parent  20a16b9bac50d7c66e599a2f5f0447764468bb7f
manifestmerge: drop redundant flags calls

Patch

diff --git a/mercurial/merge.py b/mercurial/merge.py
--- a/mercurial/merge.py
+++ b/mercurial/merge.py
@@ -275,7 +275,7 @@  def manifestmerge(repo, wctx, p2, pa, br
             pass
         elif n1 and f in movewithdir: # directory rename
             f2 = movewithdir[f]
-            actions.append((f, "d", (None, f2, m1.flags(f)),
+            actions.append((f, "d", (None, f2, fl1),
                             "remote renamed directory to " + f2))
         elif n1 and f in copy:
             f2 = copy[f]
@@ -290,7 +290,7 @@  def manifestmerge(repo, wctx, p2, pa, br
                 actions.append((f, "r", None, "other deleted"))
         elif n2 and f in movewithdir:
             f2 = movewithdir[f]
-            actions.append((None, "d", (f, f2, m2.flags(f)),
+            actions.append((None, "d", (f, f2, fl2),
                             "local renamed directory to " + f2))
         elif n2 and f in copy:
             f2 = copy[f]
@@ -314,7 +314,7 @@  def manifestmerge(repo, wctx, p2, pa, br
             # Checking whether the files are different is expensive, so we
             # don't do that when we can avoid it.
             if force and not branchmerge:
-                actions.append((f, "g", (m2.flags(f),), "remote created"))
+                actions.append((f, "g", (fl2,), "remote created"))
             else:
                 different = _checkunknownfile(repo, wctx, p2, f)
                 if force and branchmerge and different:
@@ -323,7 +323,7 @@  def manifestmerge(repo, wctx, p2, pa, br
                 elif not force and different:
                     aborts.append((f, "ud"))
                 else:
-                    actions.append((f, "g", (m2.flags(f),), "remote created"))
+                    actions.append((f, "g", (fl2,), "remote created"))
         elif n2 and n2 != ma[f]:
             prompts.append((f, "dc")) # prompt deleted/changed