Re: [PATCH 7/7] xdiff: make diff3 the default conflictStyle

2 messages, 2 authors, 2021-06-11 · open the first message on its own page

Re: [PATCH 7/7] xdiff: make diff3 the default conflictStyle

From: Junio C Hamano <hidden>
Date: 2021-06-11 07:14:49

Johannes Sixt [off-list ref] writes:
The case that inner conflicts are presented sub-optimally under diff3
remains, though.
I agree that until that happens (necessary but not sufficient
condition), it is premature to recommend diff3 style to be the
default.

I notice that "git merge --help" tells what each part separated by
conflict markers mean in both output styles, but does not make a
specific recommendation as to which one to use in what situation,
and it might benefit a few additional sentences to help readers
based on what you said, i.e. the "RCS merge" style that hides the
original is succinct and easier to work with when you are familiar
with what both sides did, while a more verbose "diff3" style helps
when you are unfamiliar with what one side (or both sides) did.

Re: [PATCH 7/7] xdiff: make diff3 the default conflictStyle

From: Felipe Contreras <hidden>
Date: 2021-06-11 14:29:48

Junio C Hamano wrote:
Johannes Sixt [off-list ref] writes:
quoted
The case that inner conflicts are presented sub-optimally under diff3
remains, though.
I agree that until that happens (necessary but not sufficient
condition), it is premature to recommend diff3 style to be the
default.
Why? Most experienced git developers have no issue with diff3. So
presumably it's good enough as it is.

Yes, we should consider improving that issue stated above, but
*necessary*? I don't think so.

-- 
Felipe Contreras
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help