( )⚙ D7349 index: use `index.get_rev` in `rebase.successorrevs`

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D7349

index: use `index.get_rev` in `rebase.successorrevs`
ClosedPublic

Authored by marmoute on Nov 8 2019, 11:26 AM.

Details

Reviewers
martinvonz
indygreg
Group Reviewers
hg-reviewers
Commits
rHG054846d213ba: index: use `index.get_rev` in `rebase.successorrevs`
rHG9167a2912721: index: use `index.get_rev` in `rebase.successorrevs`
rHG3e8796aa0809: index: use `index.get_rev` in `rebase.successorrevs`
Summary

The code is slightly updated in the process for simplicity.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Revision Contents
Changeset List

PathPackages
Mhgext/rebase.py (7 lines)

Diff 17903

hgext/rebase.py

b"experimental.evolution.allowdivergence=True" b"experimental.evolution.allowdivergence=True"
) )
raise error.Abort(msg % (b",".join(divhashes),), hint=h) raise error.Abort(msg % (b",".join(divhashes),), hint=h)
def successorrevs(unfi, rev): def successorrevs(unfi, rev):
"""yield revision numbers for successors of rev""" """yield revision numbers for successors of rev"""
assert unfi.filtername is None assert unfi.filtername is None
nodemap = unfi.changelog.nodemap get_rev = unfi.changelog.index.get_rev
for s in obsutil.allsuccessors(unfi.obsstore, [unfi[rev].node()]): for s in obsutil.allsuccessors(unfi.obsstore, [unfi[rev].node()]):
if s in nodemap: r = get_rev(s)
yield nodemap[s] if r is not None:
yield r
def defineparents(repo, rev, destmap, state, skipped, obsskipped): def defineparents(repo, rev, destmap, state, skipped, obsskipped):
"""Return new parents and optionally a merge base for rev being rebased """Return new parents and optionally a merge base for rev being rebased
The destination specified by "dest" cannot always be used directly because The destination specified by "dest" cannot always be used directly because
previously rebase result could affect destination. For example, previously rebase result could affect destination. For example,