comparison mercurial/scmutil.py @ 33352:967ac37f3d45

cleanupnode: do not use generator for node mapping

The 'successors' part of the mappings used of be a tuple. This avoid issue from
code consuming the generator "by mistake". For example, an extension inspecting the
mapping content used to be able to iterate over the successors mapping without
consequence.

Since the mapping are small we do not expect any performance impact we use tuple
again for this.

33351:154298576d44

            # Unfiltered repo is needed since nodes in mapping might be hidden.
            unfi = repo.unfiltered()
            isobs = unfi.obsstore.successors.__contains__
            torev = unfi.changelog.rev
            sortfunc = lambda ns: torev(ns[0])
            rels = [(unfi[n], (unfi[m] for m in s))
                    for n, s in sorted(mapping.items(), key=sortfunc)
                    if s or not isobs(n)]
            obsolete.createmarkers(repo, rels, operation=operation)
        else:
            from . import repair # avoid import cycle

33352:967ac37f3d45

            # Unfiltered repo is needed since nodes in mapping might be hidden.
            unfi = repo.unfiltered()
            isobs = unfi.obsstore.successors.__contains__
            torev = unfi.changelog.rev
            sortfunc = lambda ns: torev(ns[0])
            rels = [(unfi[n], tuple(unfi[m] for m in s))
                    for n, s in sorted(mapping.items(), key=sortfunc)
                    if s or not isobs(n)]
            obsolete.createmarkers(repo, rels, operation=operation)
        else:
            from . import repair # avoid import cycle