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

Re: [PATCH 1/2] tests: eliminate unnecessary setup test assertions

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:51:11

Possibly related (same subject, not in this thread)

Jeff King [off-list ref] writes:
Two minor complaints on git-blame; maybe somebody can point out
something clever I've missed.
  1. blame's "-L" understands patterns already.
Teaching blame to take multiple -L options has been one of many
longstanding todo item for me.  Someday.
  2. Parsing the human-readable output blame output sucks. But parsing
     --porcelain is annoyingly complex for quick-and-dirty things like
     this. It doesn't repeat the commit information per-line.
Non-repetition was quite deliberate, as the reader was expected to have
memory proportional to the number of lines in the range, but I agree it is
not friendly for quick and dirty hack.

You should be able to add a command line option that disables the early
return at the beginning of emit_one_suspect_detail() with a 5-6 lines of
patch.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help