Re: [PATCH 2/2] shortlog: add '--sort-key' and '--sort-key-regexp' options

From: Linus Torvalds <torvalds@linux-foundation.org>
Date: Jul 15, 2009, 15:11 UTC
Message-ID: <alpine.LFD.2.01.0907150807401.13838@localhost.localdomain>
In-Reply-To: <alpine.DEB.1.00.0907151316320.4410@intel-tinevez-2-302>

On Wed, 15 Jul 2009, Johannes Schindelin wrote:

> 
> Instead of always sorting by author, allow to sort by either
> a whitespace-delimited field or by a regular expression (first group)
> on the oneline.

Ok, I got a SIGSEGV on the regexp, because I didn't react to the "first group" part. So I did

	git shortlog --pretty=%cD --sort-key-regex ' [0-9]*:' -ns

and it immediately segfaulted for me.

I realize that I _should_ have done

	git shortlog --pretty=%cD --sort-key-regex ' \([0-9]*\):' -ns

and then I get the expected hourly statistics, but it shouldn't have SIGSEGV'd either.

But cool small patch-series. This was exactly what I was looking for.

		Linus