Map long for-loop control variable names in shaders.

Look at TIntermLoop::traverse() in IntermTraverse.cpp, the control init part is not handled explicitly there (unlike the other two parts). It is expected hat visitLoop will cover the init part. The bug in the MapLongVariableNames is that the visitLoop doesn't do anything. This CL fixes it. BUG=171 TEST=conformance/glsl-long-variable-names.html passing. Review URL: http://codereview.appspot.com/4644045 git-svn-id: https://angleproject.googlecode.com/svn/trunk@696 736b8ea6-26fd-11df-bfd4-992fa37f6226

Map long for-loop control variable names in shaders.
216aa5ed · zmo@google.com · dc4b4f85 · 216aa5ed · 216aa5ed
Commit 216aa5ed authored Jun 17, 2011 by zmo@google.com
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

version.h src/common/version.h +1 -1

MapLongVariableNames.cpp src/compiler/MapLongVariableNames.cpp +4 -2

No files found.
--- a/src/common/version.h
+++ b/src/common/version.h
 #define MAJOR_VERSION 0
 #define MINOR_VERSION 0
 #define BUILD_VERSION 0
-#define BUILD_REVISION 695
+#define BUILD_REVISION 696

 #define STRINGIFY(x) #x
 #define MACRO_STRINGIFY(x) STRINGIFY(x)

--- a/src/compiler/MapLongVariableNames.cpp
+++ b/src/compiler/MapLongVariableNames.cpp
@@ -67,13 +67,15 @@ bool MapLongVariableNames::visitSelection(Visit, TIntermSelection*)
    return true;
 }

-bool MapLongVariableNames::visitAggregate(Visit, TIntermAggregate* node)
+bool MapLongVariableNames::visitAggregate(Visit, TIntermAggregate*)
 {
    return true;
 }

-bool MapLongVariableNames::visitLoop(Visit, TIntermLoop*)
+bool MapLongVariableNames::visitLoop(Visit, TIntermLoop* node)
 {
+    if (node->getInit())
+        node->getInit()->traverse(this);
    return true;
 }