On 01/06/2016 12:32 AM, Paul Eggert wrote:
I installed the attached patch, which fixed this performance bug for me.

Whoops! I forgot to 'git add src/search.h' before committing. We also need the attached followup patch, which I installed.
>From 5a71d9d4afc2ec1a7a2c6e5c3fac33709ddc6551 Mon Sep 17 00:00:00 2001
From: Paul Eggert <egg...@cs.ucla.edu>
Date: Wed, 6 Jan 2016 09:55:28 -0800
Subject: [PATCH] grep: restore -P optimization (followup fix)

* src/search.h (EGexecute, Fexecute, Pexecute):
Change decls to match new implementations.
I forgot to add this file to the previous commit.
---
 src/search.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/search.h b/src/search.h
index 5031d67..a69bf19 100644
--- a/src/search.h
+++ b/src/search.h
@@ -57,15 +57,15 @@ extern wint_t mb_next_wc (char const *, char const *);
 
 /* dfasearch.c */
 extern void GEAcompile (char const *, size_t, reg_syntax_t);
-extern size_t EGexecute (char const *, size_t, size_t *, char const *);
+extern size_t EGexecute (char *, size_t, size_t *, char const *);
 
 /* kwsearch.c */
 extern void Fcompile (char const *, size_t);
-extern size_t Fexecute (char const *, size_t, size_t *, char const *);
+extern size_t Fexecute (char *, size_t, size_t *, char const *);
 
 /* pcresearch.c */
 extern void Pcompile (char const *, size_t);
-extern size_t Pexecute (char const *, size_t, size_t *, char const *);
+extern size_t Pexecute (char *, size_t, size_t *, char const *);
 
 /* Return the number of bytes in the character at the start of S, which
    is of size N.  N must be positive.  MBS is the conversion state.
-- 
2.5.0

Reply via email to