Re: [PATCH 00/21] nd/diff-parseopt part 2

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 11, 2019, 17:12 UTC
Message-ID: <xmqqo97ii714.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20190207184300.GN10587@szeder.dev>

SZEDER Gábor <szeder.dev@gmail.com> writes:

Show 14 quoted lines

> On Thu, Feb 07, 2019 at 05:33:05PM +0700, Nguyễn Thái Ngọc Duy wrote:
>> Nguyễn Thái Ngọc Duy (21):
>>   diff.c: convert --patch-with-raw
>>   diff.c: convert --numstat and --shortstat
>>   diff.c: convert --dirstat and friends
>>   diff.c: convert --check
>>   ...
>>   diff.c: convert --no-renames|--[no--rename-empty
>>   diff.c: convert --relative
>>   diff.c: convert --[no-]minimal
>>   diff.c: convert --ignore-some-changes
>
> Nit: convert to what?  Perhaps a 's/$/ to parse-options/' would
> improve the shortlog/oneline output.

But that would be hidden by getting pushed far to the right.

I am wondering if we can do something clever with the <area>: prefix. For a series like this one, the shortlog output may be better off if a short token that clearly identifies the topic is used there, e.g.

    diff-parseopt: convert --patch-with-raw
    diff-parseopt: convert --numstat and --shortstat
    ...

or even

    diff-parseopt: --patch-with-raw

perhaps.