This is an archive of the discontinued Mercurial Phabricator instance.

rebase: avoid defining two lists with the same contents
ClosedPublic

Authored by martinvonz on Mar 16 2018, 1:36 AM.

Details

Summary

In abort(), there's "dstates" and "rebased" that are identical, which
they seem to have been since 0806823370d8 (rebase: properly calculate
descendant set when aborting (issue3332), 2012-03-22). Let's
de-duplicate. I don't know what "dstates" means, but "rebased" makes
sense (it's the list of rebased revisions), so let's pick that.

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.

Event Timeline

martinvonz created this revision.Mar 16 2018, 1:36 AM
yuja accepted this revision.Mar 19 2018, 10:09 AM
This revision is now accepted and ready to land.Mar 19 2018, 10:09 AM
This revision was automatically updated to reflect the committed changes.