Re: [PATCH] Remove arrays in usage text, use a single string instead

[Denys Vlasenko]

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.
-- 
vda