On Tue, Nov 3, 2015 at 9:27 AM, Eric Blake <ebl...@redhat.com> wrote:
> On 11/03/2015 10:05 AM, Giuseppe Scrivano wrote:
>
>> I have attached the patches that implement --palette, the missing tests
>> and update the NEWS file.
>>
>
>> +++ b/doc/diffutils.texi
>> @@ -3763,6 +3763,7 @@ Always use color.
>> Specifying @option{--color} and no @var{when} is equivalent to
>> @option{--color=auto}.
>>
>> +
>> @item -C @var{lines}
>
> Spurious change?
>
>> @itemx --context@r{[}=@var{lines}@r{]}
>> Use the context output format, showing @var{lines} (an integer) lines of
>> @@ -3890,6 +3891,11 @@ if-then-else format. @xref{Line Formats}.
>> @itemx --show-c-function
>> Show which C function each change is in. @xref{C Function Headings}.
>>
>> +@item --palette=@var{scheme}
>> +It allows to specify what colors are used to colorize the output. It
>
> Passive voice. Would sound better as:
>
> Specify what color palette to use when colored output to use.
Thanks for the quick review, Eric.
I'll wait for the next iteration.
