From: Junio C Hamano <gitster@pobox.com>
Date: Sat, 21 Mar 2020 20:11:28 GMT
Subject: Re: [PATCH] test-lib: allow short options to be stacked
Message-ID: <xmqqpnd58o3z.fsf@gitster.c.googlers.com>
In-Reply-To: <CAHd-oW73xp2=XcFr1F57fo2UtAyuZ2aeXVSc_A50bYKKVN7fJw@mail.gmail.com>

Matheus Tavares Bernardino <matheus.bernardino@usp.br> writes:

> Makes sense, thanks. However, using Peff's suggestion[1] for the
> character iteration already eliminates the need for extra processes.

Even without an extra process, having to strip "-", assign an empty
string to $extra, and then strip that empty string from the tail to
come up with a single letter in $this, all are consuming cycles.
Even though these wasted cycles are now much smaller, having an arm
that specifically catches unbundled case and avoid doing anything
extra makes the codeflow clear, I would think.