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

diff --git a/src/ChangeLog b/src/ChangeLog
index 893ad867c2a83219b1b6640fc3a4bf2ab9282c1b..af53b77a62267f5ef118392a00e94342d1cde874 100644 (file)
--- 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 67eb59e7899c4fe3a19ea6e0eec6f42c226adf9a..bbafc785e7bb8cc6efbaa8614d98d2b4b694db6f 100644 (file)
--- 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);