darcs

Patch 1608 refactor tentativelyMergePatches_ (6 patches)

Title refactor tentativelyMergePatches_ (6 patches)
Superseder Nosy List bf
Related Issues
Status accepted Assigned To
Milestone

Created on 2017-10-03.17:29:54 by bf, last changed 2017-10-09.15:52:08 by gh.

Files
File name Status Uploaded Type Edit Remove
moved-makechanges-from-d_r_hashed-to-d_r_merge.dpatch bf, 2017-10-03.17:29:53 application/x-darcs-patch
See mailing list archives for discussion on individual patches.
Messages
msg19692 (view) Author: bf Date: 2017-10-03.17:29:53
The result is functionally identical but is now easier to read and
understand and also more efficient (with --reorder-patches).
Attachments
msg19714 (view) Author: gh Date: 2017-10-09.15:52:07
Everything looks correct, except the reorder code which I have trouble
to go through every step in my available time, but for this code I'm
going to trust the (passing) shell test.

Note that after this bundle, tentativelyReplacePatches is no longer used
outside of src/Darcs/Repository/Hashed.hs .

Accepted.
History
Date User Action Args
2017-10-03 17:29:54bfcreate
2017-10-03 19:04:05bfsetstatus: needs-screening -> needs-review
2017-10-09 15:52:08ghsetstatus: needs-review -> accepted
messages: + msg19714