Hi Eric,
> Le 28 oct. 2020 à 13:33, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> a écrit :
>
> On Wed, Oct 28, 2020 at 8:48 AM Philippe Blain via GitGitGadget
> <gitgitgadget@xxxxxxxxx> wrote:
>> 'git blame -h' and 'git log -h' both show '-L <n,m>' and describe this
>> option as "Process only line range n,m, counting from 1". No hint is
>> given that a function name regex can also be used.
>>
>> Use <range> instead, and expand the description of the option to mention
>> both modes. Remove "counting from 1" as it's uneeded; it's uncommon to
>> refer to the first line of a file as "line 0".
>>
>> Signed-off-by: Philippe Blain <levraiphilippeblain@xxxxxxxxx>
>> ---
>> diff --git a/builtin/blame.c b/builtin/blame.c
>> @@ -889,7 +889,8 @@ int cmd_blame(int argc, const char **argv, const char *prefix)
>> - OPT_STRING_LIST('L', NULL, &range_list, N_("n,m"), N_("Process only line range n,m, counting from 1")),
>> + OPT_STRING_LIST('L', NULL, &range_list, N_("range"),
>> + N_("Process only the given line range (<range>=<start>,<end>) or function (<range>=:<funcname>)")),
>
> The "<range>=" bit is redundant and confusing (and ugly). Considering
> that the description already says "Process only the given line
> _range_", it should be fine to drop the "<range>=" lead-in. Perhaps:
>
> Process only the given line range <start>,<end> or :<funcname>"
>
> This might feel too succinct, but that's often true of short -h help.
> Such succinctness is generally acceptable as long as more detailed
> documentation can be discovered easily (such as in the 'man' page).
>
> Same comment regarding the rest of the changes in this patch.
Yes, I agree. I've shortened it for v2 (and also made the changes you suggested
for 2/6 and 4/6.
Thanks for taking a look,
Philippe.
