Elijah Newren [off-list ref] writes:
On Mon, Aug 22, 2022 at 7:42 PM Elijah Newren via GitGitGadget
[off-list ref] wrote:
quoted
While working on other things, I noticed a few miscellaneous issues in
builtin/merge.c. Here's a few small fixes to address them.
Sorry, forgot to include a high level summary:
Changes since v1:
* Fix play/apply typo, spotted by Eric
* Almost completely rewrote patch 2; it's now merely a "code cleanup"
* Added a third patch, just a very minor code simplification
I was skeptical about patch 2 before I saw it, but removal of
!merge_was_ok in the loop condition alone is very well worth it.
As to patch 3, I do not think it makes huge difference either way,
but I've queued it anyway.
Thanks.