commit 7351c2a59f8c83aeb385895462167c09bea4fb89
author Olli Etuaho <oetuaho@nvidia.com> Thu Dec 15 18:06:41 2016 +0000
committer Commit Bot <commit-bot@chromium.org> Fri Dec 16 15:53:31 2016 +0000
tree 3dcb3fe74370699d3110a82a34f8e2bf35471091
parent 439a55e55038b357d76f306cfa2e034e3b93fe5e

Clean up GLSL switch statement validation

Encapsulate all of the implementation inside the .cpp file, and pass
just the diagnostics object instead of the whole ParseContext to the
validation function.

BUG=angleproject:1670
TEST=angle_unittests

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

src/compiler/translator/ParseContext.cpp

diff --git a/src/compiler/translator/ParseContext.cpp b/src/compiler/translator/ParseContext.cpp
index d9618df..30e8b9d 100644
--- a/src/compiler/translator/ParseContext.cpp
+++ b/src/compiler/translator/ParseContext.cpp
@@ -3542,7 +3542,7 @@
 
     if (statementList)
     {
-        if (!ValidateSwitch::validate(switchType, this, statementList, loc))
+        if (!ValidateSwitchStatementList(switchType, &mDiagnostics, statementList, loc))
         {
             return nullptr;
         }