On 10/08/2010 04:20 PM, Denys Vlasenko wrote:
On Fri, 2010-10-08 at 14:23 +0200, Nikola Pajkovsky wrote:
> On 10/08/2010 02:10 PM, Denys Vlasenko wrote:
>> Currently, usage is an array of pointers to string.
>> It is meant to support this:
>>
>> usage: foo --bar
>> or: foo --baz
>> or: foo -z
>>
>> The patch removes this and makes usage just a string:
>>
>> usage: foo --bar
>>
>> Why remove this?
>>
>> (1) We can achieve old format by using the usage string with many lines:
>> usage = "foo --bar\n"
>> " or: foo --baz\n"
>> " or: foo -z";
>>
>> (2) We don't use multi-line usage anyway.
>>
>> Run tested.
>>
>> Please review.
>
> +#define OPT__VERBOSE(v) OPT_BOOL('v', "verbose", (v), "Be
verbose")
>
> Why upper 'B' in `Be verbose'?
>
> First I don't see in any helps, second it looks bad and third I don't like
it.
I committed the patch sans this capitalization change.
Thank you!
--
Nikola