src/bidi.c (bidi_find_bracket_pairs): Avoid a loop that does nothing useful.

2 files changed, 10 insertions, 2 deletions

diff --git a/src/ChangeLog b/src/ChangeLog
index 893ad867c2a..af53b77a622 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,8 @@
+2014-10-16  Eli Zaretskii  <eliz@gnu.org>
+
+        * bidi.c (bidi_find_bracket_pairs): Avoid a loop that does nothing
+        useful.
+
 2014-10-15  Paul Eggert  <eggert@cs.ucla.edu>
 
         * bidi.c (bidi_find_bracket_pairs): Initialize local var.
diff --git a/src/bidi.c b/src/bidi.c
index 67eb59e7899..bbafc785e7b 100644
--- a/src/bidi.c
+++ b/src/bidi.c
@@ -2472,8 +2472,11 @@ bidi_find_bracket_pairs (struct bidi_it *bidi_it)
                 default:
                   break;
                 }
-              for (sp = bpa_sp; sp >= 0; sp--)
-                bpa_stack[sp].flags |= flag;
+              if (flag)
+                {
+                  for (sp = bpa_sp; sp >= 0; sp--)
+                    bpa_stack[sp].flags |= flag;
+                }
             }
           old_sidx = bidi_it->stack_idx;
           type = bidi_resolve_weak (bidi_it);