Re: [PATCH] shortlog: skip format/parse roundtrip for internal traversal

2017-09-10 Thread Jeff King
On Sun, Sep 10, 2017 at 10:44:46AM +0200, René Scharfe wrote: > Am 08.09.2017 um 11:21 schrieb Jeff King: > > Note that the non-stdin path no longer looks at the "mailmap" entry of > > "struct shortlog" (instead we use the one cached inside pretty.c). But > > we still waste time loading it. I'm no

Re: [PATCH] shortlog: skip format/parse roundtrip for internal traversal

2017-09-10 Thread René Scharfe
Am 08.09.2017 um 11:21 schrieb Jeff King: > Note that the non-stdin path no longer looks at the "mailmap" entry of > "struct shortlog" (instead we use the one cached inside pretty.c). But > we still waste time loading it. I'm not sure if it's worth addressing > that. It's only once per program invo

[PATCH] shortlog: skip format/parse roundtrip for internal traversal

2017-09-08 Thread Jeff King
On Fri, Sep 08, 2017 at 03:39:36PM +0900, Junio C Hamano wrote: > Jeff King writes: > > > IOW, something like the patch below, which pushes the re-parsing out to > > the stdin code-path, and lets the internal traversal format directly > > into the final buffer. It seems to be about 3% faster tha