Re: [patch] selected/context colors

[Benno Schulenberg]

Charles Levert wrote:
> Please check that all colors as well as the various
> separators are ok.

Phew...  I can't imagine ever wanting to use all those colours.  
That the file name is now different from the file contents, that's 
pleasing.  But when selected lines differ from context lines, it 
already becomes carnival: more confusing than helpful.

But the colourings that I'm likely appear to use work fine: file 
name, line number, matched text.

BTW, when using --color=always and piping the output to 'less', text 
in a line containing a coloured word cannot be searched and found.  
Just a note.

> whether -v (or not -v). 

"whether or not with -v".

>  /* Other colors.  Defaults look damn good.  */

Please remove the strong word.

"The defaults are pretty good."

Benno