scmutil: obsrevs is already a frozenset
authorAnton Shestakov <av6@dwimlabs.net>
Mon, 07 Feb 2022 12:29:43 +0300
changeset 48724 c7e675848027
parent 48723 27fe84a8dd60
child 48728 3199b575375d
scmutil: obsrevs is already a frozenset Differential Revision: https://phab.mercurial-scm.org/D12157
mercurial/scmutil.py
--- a/mercurial/scmutil.py	Mon Feb 07 12:25:46 2022 +0300
+++ b/mercurial/scmutil.py	Mon Feb 07 12:29:43 2022 +0300
@@ -366,9 +366,7 @@
         obsrevs = obsolete.getrevs(repo, b'obsolete')
         if not cl.filteredrevs and not obsrevs:
             return None
-        # TODO: obsrevs should be a frozenset, but right now obsolete.getrevs()
-        # may return a set, which is not a hashable type.
-        key = (maxrev, hash(cl.filteredrevs), hash(frozenset(obsrevs)))
+        key = (maxrev, hash(cl.filteredrevs), hash(obsrevs))
     else:
         if not cl.filteredrevs:
             return None