Fix a traverse bug.

In loop node, the init part was skipped in traversing. BUG= TEST= R=kbr@chromium.org Review URL: https://codereview.appspot.com/14366043 Change-Id: If3200f1dbcafda1147820b2d47b758b897de0fc6 Reviewed-on: https://chromium-review.googlesource.com/178992Reviewed-by: Shannon Woods <shannonwoods@chromium.org> Tested-by: Shannon Woods <shannonwoods@chromium.org>

Fix a traverse bug.
6cb95f3a · Zhenyao Mo · Shannon Woods · e88dcaf3 · 6cb95f3a · 6cb95f3a
Commit 6cb95f3a authored Oct 03, 2013 by Zhenyao Mo Committed by Shannon Woods Dec 09, 2013
3 changed files
--- a/src/compiler/translator/IntermTraverse.cpp
+++ b/src/compiler/translator/IntermTraverse.cpp
@@ -211,9 +211,15 @@ void TIntermLoop::traverse(TIntermTraverser *it)

            if (cond)
                cond->traverse(it);
+
+            if (init)
+                init->traverse(it);
        }
        else
        {
+            if (init)
+                init->traverse(it);
+
            if (cond)
                cond->traverse(it);


--- a/src/compiler/translator/MapLongVariableNames.cpp
+++ b/src/compiler/translator/MapLongVariableNames.cpp
@@ -102,13 +102,6 @@ void MapLongVariableNames::visitSymbol(TIntermSymbol* symbol)
    }
 }

-bool MapLongVariableNames::visitLoop(Visit, TIntermLoop* node)
-{
-    if (node->getInit())
-        node->getInit()->traverse(this);
-    return true;
-}
-
 TString MapLongVariableNames::mapGlobalLongName(const TString& name)
 {
    ASSERT(mGlobalMap);

--- a/src/compiler/translator/MapLongVariableNames.h
+++ b/src/compiler/translator/MapLongVariableNames.h
@@ -48,7 +48,6 @@ public:
    MapLongVariableNames(LongNameMap* globalMap);

    virtual void visitSymbol(TIntermSymbol*);
-    virtual bool visitLoop(Visit, TIntermLoop*);

 private:
    TString mapGlobalLongName(const TString& name);