D4411: obsutil: make sure "addedmarkers" is not None in getobsoleted()

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Tue Aug 28 22:30:27 UTC 2018


martinvonz created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  Similar reasoning as in https://phab.mercurial-scm.org/rHG52e6171ec822165d07cd8fdad3dd10aa8af8990f (obsutil: make sure "addedrevs"
  is not None in getobsoleted(), 2018-08-25).

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  mercurial/obsutil.py

CHANGE DETAILS

diff --git a/mercurial/obsutil.py b/mercurial/obsutil.py
--- a/mercurial/obsutil.py
+++ b/mercurial/obsutil.py
@@ -464,7 +464,7 @@
     phase = repo._phasecache.phase
     succsmarkers = repo.obsstore.successors.get
     public = phases.public
-    addedmarkers = tr.changes.get('obsmarkers')
+    addedmarkers = tr.changes['obsmarkers']
     origrepolen = tr.changes['origrepolen']
     seenrevs = set()
     obsoleted = set()



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


More information about the Mercurial-devel mailing list