diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2014-10-24 13:27:29 -0700 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2014-10-24 13:35:19 -0700 |
| commit | 16f6616acaa4fcf44edbc3b56ca10fc06b07cf25 (patch) | |
| tree | 7c29ee4edb115141168c73fa7a8eccdc2ff6e9af /src/grep.h | |
| parent | b2490802defe3c3bf7ef0036a4515d006a08a769 (diff) | |
| download | grep-16f6616acaa4fcf44edbc3b56ca10fc06b07cf25.tar.gz | |
grep: fix grep -P crash
Reported by Shlomi Fish in: http://bugs.gnu.org/18806
Commit 9fa500407137f49f6edc3c6b4ee6c7096f0190c5 (2014-09-16) is a
hack that I put in to speed up 'grep -P'. Unfortunately, not only
is it violation of modularity, it's also a bug magnet, as we have
found out with Bug#18738 and Bug#18806. Remove the optimization
instead of applying more bandaids. Perhaps we can think of a
better way of doing the optimization, or perhaps we can just live
with a slower grep -P (as -P is inherently slower anyway...).
* src/grep.c, src/grep.h (validated_boundary):
Remove. All uses removed.
* src/pcresearch.c (Pexecute): Do not worry about validated_boundary.
Diffstat (limited to 'src/grep.h')
| -rw-r--r-- | src/grep.h | 4 |
1 files changed, 0 insertions, 4 deletions
@@ -47,8 +47,4 @@ enum textbin /* Input file type. */ extern enum textbin input_textbin; -/* Validation boundary. Earlier bytes have already been validated by - the PCRE matcher, which cares about this sort of thing. */ -extern char const *validated_boundary; - #endif |