commit d05f964b1abd8821ee6324909138720cdac337a1
author Olli Etuaho <oetuaho@nvidia.com> Mon Mar 05 12:13:26 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Mar 05 16:07:00 2018 +0000
tree 2d018b4adea1d24512cbb33ebda5148468ee92a6
parent dc0cdba88ae36c61e59c2f9f109f55f7c0110d0e

Last case in switch statement can't be empty in ESSL 3.10

This is based on recent discussion in Khronos, though public specs
have not yet been updated to reflect this.

BUG=angleproject:2388
TEST=angle_unittests

Change-Id: I66a0d03b3c2bb9740772a813b543f8f6c6bb2a28
Reviewed-on: https://chromium-review.googlesource.com/947977
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 25b6fd9..fdc7652 100644
--- a/src/compiler/translator/ParseContext.cpp
+++ b/src/compiler/translator/ParseContext.cpp
@@ -4840,7 +4840,7 @@
     }
 
     ASSERT(statementList);
-    if (!ValidateSwitchStatementList(switchType, mShaderVersion, mDiagnostics, statementList, loc))
+    if (!ValidateSwitchStatementList(switchType, mDiagnostics, statementList, loc))
     {
         ASSERT(mDiagnostics->numErrors() > 0);
         return nullptr;