D5849: merge: don't unnecessarily calculate absolute path
martinvonz (Martin von Zweigbergk)
phabricator at mercurial-scm.org
Tue Feb 5 13:13:38 EST 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rHG1f2714052d7e: merge: don't unnecessarily calculate absolute path (authored by martinvonz, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D5849?vs=13810&id=13827
REVISION DETAIL
https://phab.mercurial-scm.org/D5849
AFFECTED FILES
mercurial/merge.py
CHANGE DETAILS
diff --git a/mercurial/merge.py b/mercurial/merge.py
--- a/mercurial/merge.py
+++ b/mercurial/merge.py
@@ -1499,13 +1499,14 @@
# If a file or directory exists with the same name, back that
# up. Otherwise, look to see if there is a file that conflicts
# with a directory this file is in, and if so, back that up.
- absf = repo.wjoin(f)
+ conflicting = f
if not repo.wvfs.lexists(f):
for p in util.finddirs(f):
if repo.wvfs.isfileorlink(p):
- absf = repo.wjoin(p)
+ conflicting = p
break
- if repo.wvfs.lexists(absf):
+ if repo.wvfs.lexists(conflicting):
+ absf = repo.wjoin(conflicting)
orig = scmutil.origpath(ui, repo, absf)
util.rename(absf, orig)
wctx[f].clearunknown()
To: martinvonz, #hg-reviewers, pulkit
Cc: pulkit, mercurial-devel
More information about the Mercurial-devel
mailing list