durin42 edited the summary of this revision. durin42 retitled this revision from "branchmap: always copy closednodes to a set" to "branchmap: correctly set()-ify list argument". durin42 updated this revision to Diff 18077.
REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D7289?vs=18063&id=18077 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7289/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7289 AFFECTED FILES mercurial/branchmap.py CHANGE DETAILS diff --git a/mercurial/branchmap.py b/mercurial/branchmap.py --- a/mercurial/branchmap.py +++ b/mercurial/branchmap.py @@ -105,7 +105,7 @@ remotebranchmap, repo[rtiprev].node(), rtiprev, - closednodes=closed, + closednodes=set(closed), ) # Try to stick it as low as possible @@ -177,7 +177,7 @@ if closednodes is None: self._closednodes = set() else: - self._closednodes = closednodes + self._closednodes = set(closednodes) self._entries = dict(entries) # whether closed nodes are verified or not self._closedverified = False To: durin42, #hg-reviewers, indygreg, dlax Cc: indygreg, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel