Thread (1 message) 1 message, 1 author, 2022-08-24

Re: [PATCH v2 0/3] Miscellaneous merge fixes

From: Junio C Hamano <hidden>
Date: 2022-08-24 21:09:12

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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help