Bruno Haible wrote:
I think you can now remove the 'invert = true;' statement (line 1022).
Thanks, done with the attached.
>From 89683adbc0b4c6c75727f25dcd417f0b15f3517f Mon Sep 17 00:00:00 2001
From: Paul Eggert <egg...@cs.ucla.edu>
Date: Mon, 2 Jan 2017 08:27:12 -0800
Subject: [PATCH] dfa: remove duplicate assignment

Problem reported by Bruno Haible in:
http://lists.gnu.org/archive/html/bug-gnulib/2017-01/msg00007.html
* lib/dfa.c (parse_bracket_exp): Simplify.
---
 ChangeLog | 7 +++++++
 lib/dfa.c | 1 -
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 2b8cbc1..6ee2380 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2017-01-02  Paul Eggert  <egg...@cs.ucla.edu>
+
+	dfa: remove duplicate assignment
+	Problem reported by Bruno Haible in:
+	http://lists.gnu.org/archive/html/bug-gnulib/2017-01/msg00007.html
+	* lib/dfa.c (parse_bracket_exp): Simplify.
+
 2017-01-01  Paul Eggert  <egg...@cs.ucla.edu>
 
 	dfa: simplify constraint-dependency checking
diff --git a/lib/dfa.c b/lib/dfa.c
index cd600aa..6f0a12f 100644
--- a/lib/dfa.c
+++ b/lib/dfa.c
@@ -1019,7 +1019,6 @@ parse_bracket_exp (struct dfa *dfa)
   if (invert)
     {
       FETCH_WC (dfa, c, wc, _("unbalanced ["));
-      invert = true;
       known_bracket_exp = dfa->simple_locale;
     }
 
-- 
2.7.4

Reply via email to