Re: [PATCH 0/6] grep with textconv

2013-04-20 Thread Michael J Gruber
Junio C Hamano venit, vidit, dixit 19.04.2013 20:24: > Michael J Gruber writes: > >> This series teaches show and grep to obey textconv: show by >> default (like diff), grep only on request (--textconv). We might >> switch the default for the latter also, of course. I'd actually >> like that. >

Re: [PATCH 0/6] grep with textconv

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 11:24:16AM -0700, Junio C Hamano wrote: > > grep: allow to use textconv filters > > This looked mostly sensible except for one minor "eh, do we really > need to assume textconv output is text, or wouldn't using the same > codepath for raw blob and textconv result to make

Re: [PATCH 0/6] grep with textconv

2013-04-19 Thread Junio C Hamano
Michael J Gruber writes: > This series teaches show and grep to obey textconv: show by > default (like diff), grep only on request (--textconv). We might > switch the default for the latter also, of course. I'd actually > like that. > > Compared to an earlier (historic) series this one comes wi

[PATCH 0/6] grep with textconv

2013-04-19 Thread Michael J Gruber
This series teaches show and grep to obey textconv: show by default (like diff), grep only on request (--textconv). We might switch the default for the latter also, of course. I'd actually like that. Compared to an earlier (historic) series this one comes with tests. Besides, it has been in use si