On Thu, Mar 10, 2022 at 1:26 PM Junio C Hamano <gitster@pobox.com> wrote:
>
> 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" (

I was "ok, that's really easy" and did

+               OPT_ALIAS(0, "new-only", "no-output-indicator-old"),
+               OPT_ALIAS(0, "old-only", "no-output-indicator-new"),

but sadly the parse-options alias code isn't quite smart enough.

Doing it with an explicit callback obviously works, but the "unique
abbreviations" part doesn't actually work for me. I think it's due to
PARSE_OPT_KEEP_UNKNOWN making the abbreviated options not work, but I
don't know tha option parsing code well enough.

Here's the stupid patch that "works" but doesn't allow the shortened
version. Maybe somebody can point out what silly thing I did wrong.

               Linus