Re: [bug-grep] [PATCH] --initial-tab, 3 newly colorized items, and GREP_COLORS

[Julian Foad]

Charles Levert wrote:
> The background:  parse_grep_colors() stops processing GREP_COLORS as soon
> as it encounters something that is ill-formed, while retaining everything
> that it processed so far.  I stand by this part of the behavior, since
> it is simplest and no behavior in particular is owed when a contract
> is broken.

Agreed.

> When this happens though, the current patch stays silent about it;
> that's the alternative to printing a warning.

My preference would be to print the warning, because I really don't like errors 
to be ignored silently even if it is a non-essential feature.

- Julian