Thread (1 message) 1 message, 1 author, 2017-11-27

Re: [PATCH] grep: Add option --max-line-len

From: Junio C Hamano <hidden>
Date: 2017-11-27 02:08:08

Marc-Antoine Ruel [off-list ref] writes:
[second try, now with text format]

Thanks a lot for the reviews. Replying to both.

If I send a follow up, I'll fix the commit description and the help
string, remove the shorthand -M, write a more sensible test.

...

Thanks for the thoughtful analysis. My main motivation was (1), thus
filtering with a pathspec is much better than trying to work around
the issue. The issues raised in the review are significant enough that
committing this patch could cause significant issues; I don't know how
to resolve handling with -v and how to handle tabs.
OK.  Thanks for thinking it through.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help