I like René's approach, too. It's more flexible, supports the old behaviour and it scratches my itch as well. Don't mind if you dropped my patch and used René's instead. >> Only *very* lightly tested, and a test for t/is missing anyway. Just >> wanted to quickly show what I meant. You'd set color.grep.matchContext="" >> to turn off highlighting in context lines. What do you think? > > I didn't realize that people wanted to see pieces on non-matching > lines highlighted. It makes certain sense, e.g. it would allow you > to spot near-misses, but that is only true for lines that neighbour > real hits, so... > > I like this approach better in that it makes those who want a > different behaviour to do the work without breaking the expectation > of those who are used to the established behaviour. > > Zoltan? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html