phases: move RemotePhasesSummary to revision number This continue our quest to align more logic on revision number instead of node-ids. The motivation is similar to the change to `new_heads` and `analyze_remote_phases` a few changeset earlier. Again, we take this as an opportunity to rename the class, and the attribute to the new naming scheme. This will highlight the need for code update for any code using it an expecting node-ids. Many of the rev-num → node-id conversion we had to introduce in the previous changesets can now be removed. More will be removed in the future as we continue to align code toward rev-num usage. time saved in the 100 milliseconds order of magnitude for the mozilla-try benchmark I have been using.

     1

# Dummy extension to define a namespace containing revision names

     2

     3

     4

from mercurial import namespaces

     5

     6

     7

def reposetup(ui, repo):

     8

    names = {b'r%d' % rev: repo[rev].node() for rev in repo}

     9

    namemap = lambda r, name: names.get(name)

    10

    nodemap = lambda r, node: [b'r%d' % repo[node].rev()]

    11

    12

    ns = namespaces.namespace(

    13

        b'revnames',

    14

        templatename=b'revname',

    15

        logname=b'revname',

    16

        listnames=lambda r: names.keys(),

    17

        namemap=namemap,

    18

        nodemap=nodemap,

    19

    )

    20

    repo.names.addnamespace(ns)