Re: [PATCH] doc: merge: improve conflict presentation docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, May 6, 2023 at 4:40 PM Elijah Newren <newren@xxxxxxxxx> wrote:
>
> On Fri, May 5, 2023 at 6:58 AM Adam Johnson via GitGitGadget
> <gitgitgadget@xxxxxxxxx> wrote:
> >
[...]
> Everywhere else in the manual we have referred to these as "conflict
> markers", not "merge markers".
[...]
> If we're updating this documentation, perhaps it's time to drop the
> mention of RCS?
[...]
> I dislike the whole "yours" & "theirs" thing.  [...] I'm happy to see
> them used one less place here.
[...]
> "Using real values" [...] seems more likely to confuse them than using a
> descriptive term or even a fake hash (such as "aaaaaaa").
[...]
> The comment you added here about zdiff3 is correct in context,
[...]

I realized I also failed to note that I like the patch in general and
think it has some good cleanups, even if there are potential further
improvements to make.  Looking forward to your reroll.




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux