From: Jeff King <peff@peff.net>
Date: Mon, 09 May 2011 13:31:54 GMT
Subject: [PATCH 0/3] blame --line-porcelain
Message-ID: <20110509133153.GA10998@sigill.intra.peff.net>
In-Reply-To: <20110506222951.GA24474@sigill.intra.peff.net>

On Fri, May 06, 2011 at 06:29:51PM -0400, Jeff King wrote:

> > 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.
> 
> I tried that, and it is slightly more involved. You also need to break a
> multi-line run of lines that blame to a single suspect into its
> constituent lines. I am 75% of the way to such a patch if you are
> interested. It's not a lot of code, but it takes some refactoring of
> emit_porcelain.

It turned out to not be too bad. Here's the series.

  [1/3]: add tests for various blame formats
  [2/3]: blame: refactor porcelain output
  [3/3]: blame: add --line-porcelain output format

-Peff