commit b427920eff77de7ae9be5e4c1b8e23b99a884c1a
author Olli Etuaho <oetuaho@nvidia.com> Thu May 18 15:08:24 2017 +0300
committer Commit Bot <commit-bot@chromium.org> Fri May 19 06:06:01 2017 +0000
tree 6ce235fefc4b09a26e2716bd7a0551926eddedc8
parent bb580b1b8b124bacb44fc3de06a57ddd375bd987

Clean up ValidateMaxParameters

It doesn't need to use a traverser, since all function definitions can
be found simply by iterating over the children of the root node.

BUG=angleproject:2040
TEST=angle_unittests

Change-Id: I18a98eff9710485c0cdce73e7fffe124f7d7afb2
Reviewed-on: https://chromium-review.googlesource.com/508791
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>

src/compiler/translator/Compiler.cpp

diff --git a/src/compiler/translator/Compiler.cpp b/src/compiler/translator/Compiler.cpp
index 1bdb06b..4ea6e75 100644
--- a/src/compiler/translator/Compiler.cpp
+++ b/src/compiler/translator/Compiler.cpp
@@ -852,7 +852,7 @@
     return (mDiagnostics.numErrors() == 0);
 }
 
-bool TCompiler::limitExpressionComplexity(TIntermNode *root)
+bool TCompiler::limitExpressionComplexity(TIntermBlock *root)
 {
     TMaxDepthTraverser traverser(maxExpressionComplexity + 1);
     root->traverse(&traverser);
@@ -863,7 +863,7 @@
         return false;
     }
 
-    if (!ValidateMaxParameters::validate(root, maxFunctionParameters))
+    if (!ValidateMaxParameters(root, maxFunctionParameters))
     {
         mDiagnostics.globalError("Function has too many parameters.");
         return false;