Changeset 94954 in webkit

Timestamp:

Sep 12, 2011 6:33:56 AM (13 years ago)

Author:

abecsi@webkit.org

Message:

[Qt] Build fails after r94920 with strict compiler
​https://bugs.webkit.org/show_bug.cgi?id=67928

Reviewed by Csaba Osztrogonác.

• wtf/RedBlackTree.h:

(WTF::RedBlackTree::insert): Remove dead variables updateStart and newSubTreeRoot.

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• wtf/RedBlackTree.h (modified) (5 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2011-09-12  Andras Becsi  <andras.becsi@nokia.com>
+
+        [Qt] Build fails after r94920 with strict compiler
+        https://bugs.webkit.org/show_bug.cgi?id=67928
+
+        Reviewed by Csaba Osztrogonác.
+
+        * wtf/RedBlackTree.h:
+        (WTF::RedBlackTree::insert): Remove dead variables updateStart and newSubTreeRoot.
+
 2011-09-12  Patrick Gansterer  <paroga@webkit.org>
 

trunk/Source/JavaScriptCore/wtf/RedBlackTree.h

@@ -172 +172 @@
         x->setColor(Red);
 
-        // The node from which to start propagating updates upwards.
-        Node* updateStart = x->parent();
-
         while (x != m_root && x->parent()->color() == Red) {
             if (x->parent() == x->parent()->parent()->left()) {
@@ -184 +181 @@
                     x->parent()->parent()->setColor(Red);
                     x = x->parent()->parent();
-                    updateStart = x->parent();
                 } else {
                     if (x == x->parent()->right()) {
@@ -194 +190 @@
                     x->parent()->setColor(Black);
                     x->parent()->parent()->setColor(Red);
-                    Node* newSubTreeRoot = rightRotate(x->parent()->parent());
-                    updateStart = newSubTreeRoot->parent();
+                    rightRotate(x->parent()->parent());
                 }
             } else {
@@ -206 +201 @@
                     x->parent()->parent()->setColor(Red);
                     x = x->parent()->parent();
-                    updateStart = x->parent();
                 } else {
                     if (x == x->parent()->left()) {
@@ -216 +210 @@
                     x->parent()->setColor(Black);
                     x->parent()->parent()->setColor(Red);
-                    Node* newSubTreeRoot = leftRotate(x->parent()->parent());
-                    updateStart = newSubTreeRoot->parent();
+                    leftRotate(x->parent()->parent());
                 }
             }