Patchwork D6340: repair: reword comment about bookmarks logic

login
register
mail settings
Submitter phabricator
Date May 5, 2019, 1:35 a.m.
Message ID <28826ed3bf6b60989891fa3f4d123f7f@localhost.localdomain>
Download mbox | patch
Permalink /patch/39950/
State Not Applicable
Headers show

Comments

phabricator - May 5, 2019, 1:35 a.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGe10b8058da84: repair: reword comment about bookmarks logic (authored by durin42, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6340?vs=15003&id=15006

REVISION DETAIL
  https://phab.mercurial-scm.org/D6340

AFFECTED FILES
  mercurial/repair.py

CHANGE DETAILS




To: durin42, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/mercurial/repair.py b/mercurial/repair.py
--- a/mercurial/repair.py
+++ b/mercurial/repair.py
@@ -279,7 +279,9 @@ 
         if rev in tostrip:
             updatebm.append(m)
     newbmtarget = None
-    if updatebm: # don't compute anything is there is no bookmark to move anyway
+    # If we need to move bookmarks, compute bookmark
+    # targets. Otherwise we can skip doing this logic.
+    if updatebm:
         # For a set s, max(parents(s) - s) is the same as max(heads(::s - s)),
         # but is much faster
         newbmtarget = repo.revs('max(parents(%ld) - (%ld))', tostrip, tostrip)