This is an archive of the discontinued Mercurial Phabricator instance.

merge: move initial handling of mergeactions near to later one

Authored by pulkit on Sep 5 2020, 3:08 AM.



We build mergeactions in the beginning and use it in end. Let's build it just
before where it will be used. Helps making code much easier to understand.

Diff Detail

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

Event Timeline

pulkit created this revision.Sep 5 2020, 3:08 AM
marmoute accepted this revision.Sep 11 2020, 8:56 AM
marmoute added a subscriber: marmoute.

This kind of massive code block move in a huge function make me think we should slice it into smaller function. What do you think?

pulkit updated this revision to Diff 22638.Sep 14 2020, 7:13 AM
pulkit updated this revision to Diff 22660.Sep 17 2020, 5:39 AM
indygreg accepted this revision.Sep 17 2020, 10:10 PM
This revision is now accepted and ready to land.Sep 17 2020, 10:10 PM