Patchwork D7289: branchmap: correctly set()-ify list argument

login
register
mail settings
Submitter phabricator
Date Nov. 15, 2019, 3:36 a.m.
Message ID <a7cd29ff6b34dab66a68a8ff83b0dd23@localhost.localdomain>
Download mbox | patch
Permalink /patch/43227/
State Not Applicable
Headers show

Comments

phabricator - Nov. 15, 2019, 3:36 a.m.
Closed by commit rHG43f57b9620d2: branchmap: correctly set()-ify list argument (authored by durin42).
This revision was automatically updated to reflect the committed changes.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D7289?vs=18077&id=18114

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




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

Patch

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