[Emacs-diffs] trunk r118132: src/bidi.c (bidi_find_bracket_pairs): Avoid a loop that does nothing useful.

[Eli Zaretskii]

------------------------------------------------------------
revno: 118132
revision-id: address@hidden
parent: address@hidden
committer: Eli Zaretskii <address@hidden>
branch nick: trunk
timestamp: Thu 2014-10-16 09:55:34 +0300
message:
  src/bidi.c (bidi_find_bracket_pairs): Avoid a loop that does nothing useful.
modified:
  src/ChangeLog                  changelog-20091113204419-o5vbwnq5f7feedwu-1438
  src/bidi.c                     bidi.c-20091231194348-rm8gzug639w0dpq5-1

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2014-10-15 17:53:04 +0000
+++ b/src/ChangeLog     2014-10-16 06:55:34 +0000
@@ -1,3 +1,8 @@
+2014-10-16  Eli Zaretskii  <address@hidden>
+
+       * bidi.c (bidi_find_bracket_pairs): Avoid a loop that does nothing
+       useful.
+
 2014-10-15  Paul Eggert  <address@hidden>
 
        * bidi.c (bidi_find_bracket_pairs): Initialize local var.

=== modified file 'src/bidi.c'
--- a/src/bidi.c        2014-10-15 17:53:04 +0000
+++ b/src/bidi.c        2014-10-16 06:55:34 +0000
@@ -2472,8 +2472,11 @@
                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);