Thread (2 messages) flat view 2 messages, 2 authors, 2016-06-15

Re: [PATCH v2] blame: add a range option to -L

From: Jakub Narebski <hidden>
Date: 2016-06-15 22:48:45

Junio C Hamano [off-list ref] writes:
I wonder if it makes the interface more consistent to rewrite the
above comment like this:

	/*
	 * Allow "-L <something>,+20" to mean starting at <something>
	 * for 20 lines; "-L <something>,-5" for 5 lines ending at
	 * <something>; and "-L <something>,+-5" for 5 lines around
         * <something>.
	 */

and the match the code.
I like this.


The other approach would be to use -B <num> / -A <num> / -C [num], -<num>
convention from 'grep'... but git-blame uses -C and -<num> for other
things.

-- 
Jakub Narebski
Poland
ShadeHawk on #git
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help