Jim Meyering <jim@meyering.net> writes:

> On Tue, Nov 3, 2015 at 9:27 AM, Eric Blake <eblake@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.

sorry for taking so long, I hope the attached version is fine.

Regards,
Giuseppe