On Sun, May 21, 2017 at 1:50 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > >> Easy to review? 29 (I mean 30) patches? Are you kidding me?! >> >> As noted in v1 (<20170511091829.5634-1-avarab@xxxxxxxxx>; >> https://public-inbox.org/git/20170511091829.5634-1-avarab@xxxxxxxxx/) >> these are all doc, test, refactoring etc. changes needed by the >> subsequent "PCRE v2, PCRE v1 JIT, log -P & fixes" series. >> >> Since Junio hasn't been picking it I'm no longer sending updates to >> that patch series & waiting for this one to cook first. > > I actually do not mind a reroll that goes together with this. The > only reason why I skipped the earlier one was because I looked at > the original one, and the discussion on the reroll of this 'easy to > review' part indicated that it will be rerolled, before I got to > look at these upper layer patches. Great, now that the base of this is migrating to next, here's the second part of this. For v1 see <20170513234535.12749-1-avarab@xxxxxxxxx> (https://public-inbox.org/git/20170513234535.12749-1-avarab@xxxxxxxxx/). The only changes to the content are better if/else branching around conditional macros (but no functional changes) in the PCRE v1 JIT API patch in response to a comment by Simon Ruderich. The only other changes are trivial updates to the commit messages to account for t/perf changes made in the series this builds on. Ævar Arnfjörð Bjarmason (7): grep: don't redundantly compile throwaway patterns under threading grep: skip pthreads overhead when using one thread log: add -P as a synonym for --perl-regexp grep: add support for the PCRE v1 JIT API grep: un-break building with PCRE < 8.32 grep: un-break building with PCRE < 8.20 grep: add support for PCRE v2 Documentation/rev-list-options.txt | 1 + Makefile | 30 +++++-- builtin/grep.c | 16 +++- configure.ac | 77 +++++++++++++--- grep.c | 177 ++++++++++++++++++++++++++++++++++++- grep.h | 31 +++++++ revision.c | 2 +- t/t4202-log.sh | 12 +++ t/test-lib.sh | 2 +- 9 files changed, 324 insertions(+), 24 deletions(-) -- 2.13.0.303.g4ebf302169