Re: RFC: Using '--no-output-indicator-old' to only show new state

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 10, 2022, 21:26 UTC
Message-ID: <xmqqfsnpcxdm.fsf@gitster.g>
In-Reply-To: <CAHk-=wg+n_-btzoyMqnDYsJxFYAyUh0Kb=TkpS8RiD_h3G=_cA@mail.gmail.com>

Linus Torvalds <torvalds@linux-foundation.org> writes:

Show 6 quoted lines

> So "--new/old"? "--pre/post"?
>
> Or it could be something random, and tie it with the existing "-U"
> option, where "-U+" would be "positive side only", and "-U5-" would be
> "5 context lines, negative side only". Very dense and convenient,
> maybe not all that intuitive?

I often use -W and the above would give us a natural extension, but I agree that is a bit too dense and totally unintuitive. As we use parse-options for patch output formatting options, my pick would be "--new-only" vs "--old-only" (if there were existing options that has new/old in their names, "--preimage-only" and "--postimage-only" would also work, but that's much longer), and let the option parser accept unique abbreviations like "--new" and "--old".