diff options
| author |  Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2019-07-26 17:08:11 +0200 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2019-07-26 13:56:40 -0700 |
| commit | 04bef50c0175bba195443ea0edbf2991175cdd91 (patch) | |
| tree | a7a6f7e6fe5296e13605d0b7fed2937d448ff09c /t/t5100/empty | |
| parent | grep: use PCRE v2 for optimized fixed-string search (diff) | |
| download | tgif-04bef50c0175bba195443ea0edbf2991175cdd91.tar.xz | |
grep: remove overly paranoid BUG(...) code
Remove code that would trigger if pcre_config() or pcre2_config() was
so broken that "do we have JIT?" wouldn't return a boolean.
I added this code back in fbaceaac47 ("grep: add support for the PCRE
v1 JIT API", 2017-05-25) and then as noted in f002532784 ("grep: print
the pcre2_jit_on value", 2019-07-22) incorrectly copy/pasted some of
it in 94da9193a6 ("grep: add support for PCRE v2", 2017-06-01).
Let's just remove this code. Being this paranoid about the
pcre2?_config() function itself being broken is crossing the line into
unreasonable paranoia.
Reported-by: Beat Bolli <dev+git@drbeat.li>
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t5100/empty')
0 files changed, 0 insertions, 0 deletions