Bug 1403444 - Remove rbp_f_synced. r?njn draft
authorMike Hommey <mh+mozilla@glandium.org>
Wed, 27 Sep 2017 13:50:50 +0900
changeset 670936 c21e81f77ae1b167d9db5435bbd75ba30572c81e
parent 670935 2154d015f9f9cae085465dcf2e3e2c709a521ce6
child 670937 8662255eb3de7f8fb54f5d7c8dc3281881c437c5
child 670938 45e6d757ef72f59b9ee9294183b4cc8ad445e48f
push id81767
push userbmo:mh+mozilla@glandium.org
push dateWed, 27 Sep 2017 06:43:40 +0000
reviewersnjn
bugs1403444, 1365460
milestone58.0a1
Bug 1403444 - Remove rbp_f_synced. r?njn Bug 1365460 removed the macros that used it.
memory/build/rb.h
--- a/memory/build/rb.h
+++ b/memory/build/rb.h
@@ -664,36 +664,31 @@ private:
 #define rbp_f_height (3 * ((SIZEOF_PTR << 3) - (SIZEOF_PTR_2POW + 1)))
 
 #define rb_foreach_begin(a_type, a_field, a_tree, a_var)                       \
   {                                                                            \
     {                                                                          \
       /* Initialize the path to contain the left spine.             */         \
       a_type* rbp_f_path[rbp_f_height];                                        \
       a_type* rbp_f_node;                                                      \
-      bool rbp_f_synced = false;                                               \
       unsigned rbp_f_depth = 0;                                                \
       if ((a_tree)->rbt_root != &(a_tree)->rbt_nil) {                          \
         rbp_f_path[rbp_f_depth] = (a_tree)->rbt_root;                          \
         rbp_f_depth++;                                                         \
         while ((rbp_f_node = a_field(rbp_f_path[rbp_f_depth - 1]).Left()) !=   \
                &(a_tree)->rbt_nil) {                                           \
           rbp_f_path[rbp_f_depth] = rbp_f_node;                                \
           rbp_f_depth++;                                                       \
         }                                                                      \
       }                                                                        \
       /* While the path is non-empty, iterate.                      */         \
       while (rbp_f_depth > 0) {                                                \
         (a_var) = rbp_f_path[rbp_f_depth - 1];
 
 #define rb_foreach_end(a_type, a_field, a_tree, a_var)                         \
-  if (rbp_f_synced) {                                                          \
-    rbp_f_synced = false;                                                      \
-    continue;                                                                  \
-  }                                                                            \
   /* Find the successor.                                    */                 \
   if ((rbp_f_node = a_field(rbp_f_path[rbp_f_depth - 1]).Right()) !=           \
       &(a_tree)->rbt_nil) {                                                    \
     /* The successor is the left-most node in the right   */                   \
     /* subtree.                                           */                   \
     rbp_f_path[rbp_f_depth] = rbp_f_node;                                      \
     rbp_f_depth++;                                                             \
     while ((rbp_f_node = a_field(rbp_f_path[rbp_f_depth - 1]).Left()) !=       \