Page MenuHomePhabricator

merge: replace a repo.lookup('.') by more typical repo['.'].node()
ClosedPublic

Authored by martinvonz on Jan 24 2020, 7:16 PM.

Details

Summary

The repo.lookup('.') form comes from b3311e26f94f (merge: fix
--preview to show all nodes that will be merged (issue2043).,
2010-02-15). I don't know why that commit changed from repo['.'],
but I don't think there's any reason to do that. Note that performance
should not be a reason (anymore?), because repo.lookup() is
implemented by first creating a context object.

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.