From 959c5a309e5effaae39f0cae8889e90dbb525031 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Thu, 29 Dec 2016 23:02:45 -0800 Subject: [PATCH] dfa: minor performance tweak * lib/dfa.c (setbit_wc): Test < 0, not == EOF. --- ChangeLog | 3 +++ lib/dfa.c | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 27ee040639..6d372124b3 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2016-12-30 Paul Eggert + dfa: minor performance tweak + * lib/dfa.c (setbit_wc): Test < 0, not == EOF. + dfa: wrap charclass inside a struct On my platform (gcc Ubuntu 5.4.0-6ubuntu1~16.04.4 x86-64, en_US.utf8 locale) this makes 'grep -Fi -f list.txt list.txt >out' diff --git a/lib/dfa.c b/lib/dfa.c index a4ccf8c25a..cdc4787ec2 100644 --- a/lib/dfa.c +++ b/lib/dfa.c @@ -851,7 +851,7 @@ static bool setbit_wc (wint_t wc, charclass *c) { int b = wctob (wc); - if (b == EOF) + if (b < 0) return false; setbit (b, c); -- 2.39.5