commit 93a90fd08cd8040f7c3591a7814dbaac32fe864f
author Olli Etuaho <oetuaho@nvidia.com> Tue Apr 07 18:14:07 2015 +0300
committer Olli Etuaho <oetuaho@nvidia.com> Wed Apr 08 08:19:09 2015 +0000
tree 89646bdf72bd857954e9b987d3ebec0a2d815de4
parent 5c60031ea1117524611013f2b04217ac9ac9ca65

Remove half-baked non-sized array support

ANGLE used to accept shaders with non-sized arrays in initializer lists.
Fix this. Proper support for implicitly sized arrays will be implemented
later for ESSL3.

TEST=angle_unittests
BUG=angleproject:941

Change-Id: I53c5ccf0f7ef09ad30e142f8350812959fc4846c
Reviewed-on: https://chromium-review.googlesource.com/264360
Tested-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

src/compiler/translator/ParseContext.cpp

diff --git a/src/compiler/translator/ParseContext.cpp b/src/compiler/translator/ParseContext.cpp
index 86eeb6d..f162e42 100644
--- a/src/compiler/translator/ParseContext.cpp
+++ b/src/compiler/translator/ParseContext.cpp
@@ -1462,7 +1462,7 @@
     {
         recover();
     }
-    else if (indexExpression)
+    else
     {
         int size;
         if (arraySizeErrorCheck(arrayLocation, indexExpression, size))
@@ -1477,14 +1477,6 @@
 
         return intermediate.growAggregate(declaratorList, intermediate.addSymbol(variable ? variable->getUniqueId() : 0, identifier, type, identifierLocation), identifierLocation);
     }
-    else
-    {
-        TPublicType arrayType(publicType);
-        arrayType.setArray(true);
-        TVariable* variable = NULL;
-        if (arrayErrorCheck(arrayLocation, identifier, arrayType, variable))
-            recover();
-    }
 
     return NULL;
 }