Pranit Bauva <pranit.ba...@gmail.com> writes:
>> with the original
>>
>> case $# in
>> 0) reset to the branch ;;
>> 1) reset to the commit ;;
>> *) give usage and die ;;
>> esac
>>
>> and took the difference and reacted "ah, excess parameters are not
>> diagnosed in this function".
>>
>> Your caller does complain about excess parameters without giving
>> usage, and that is what I missed.
>>
>> I am not sure if you intended to change the behaviour in this case
>> to avoid giving the usage string; I tend to think it is a good
>> change, but I didn't see it mentioned in the proposed commit log,
>> which also contributed to my not noticing the test in the caller.
>
> I could include this in the commit message.

Advertising

Nah, it was something anybody could notice with 2 more minutes of
reading and pondering from the patch text alone. Not worth spending
more time on the log message on this one.
Thanks.