On 26/10/17 5:15 pm, Junio C Hamano wrote: > Isabella Stephens <istephens@xxxxxxxxxxxxx> writes: > >> If the -L option is used to specify a line range in git blame, and the >> end of the range is past the end of the file, git will fail with a fatal >> error. It may instead be desirable to perform a git blame for the line >> numbers in the intersection of the file and the specified line range. > > Even though erroring out upon such input was done as a deliberate > design decision, in retrospect, I do not think the design decision > made much sense. > > The code already takes a nonsense input and tries to make best sense > of it, e.g. "-L10,6" is interpreted as "-L6,10" instead of erroring > out. So if we were to do this kind of change, I suspect that it may > be better to do so unconditionally without introducing a new option. > > Thanks. > Thanks for following up. I've sent through a version 2 of the patch without the command line option.
