commit 4ddae3598ace53662bee76250ba88be51bc53812
author Olli Etuaho <oetuaho@nvidia.com> Thu Oct 26 16:20:18 2017 +0300
committer Commit Bot <commit-bot@chromium.org> Fri Oct 27 14:31:15 2017 +0000
tree 7a8abf18844100f61127de0539603162d51b7334
parent 9cd7163e11c7d2b7582190eb8987a743239bf522

Use array_specifier in struct declarator parsing

This will make implementing arrays of arrays simpler for struct
members as well. Similar refactoring was already done for other types
of array declarations.

BUG=angleproject:2125
TEST=angle_unittests

Change-Id: I0483c3a7c006d37090c7c972cb3d7763d3909c8f
Reviewed-on: https://chromium-review.googlesource.com/739824
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/compiler/translator/ParseContext.cpp

diff --git a/src/compiler/translator/ParseContext.cpp b/src/compiler/translator/ParseContext.cpp
index ed81eb3..43e0cad 100644
--- a/src/compiler/translator/ParseContext.cpp
+++ b/src/compiler/translator/ParseContext.cpp
@@ -4555,14 +4555,13 @@
 
 TField *TParseContext::parseStructArrayDeclarator(TString *identifier,
                                                   const TSourceLoc &loc,
-                                                  TIntermTyped *arraySize,
+                                                  unsigned int arraySize,
                                                   const TSourceLoc &arraySizeLoc)
 {
     checkIsNotReserved(loc, *identifier);
 
     TType *type       = new TType(EbtVoid, EbpUndefined);
-    unsigned int size = checkIsValidArraySize(arraySizeLoc, arraySize);
-    type->makeArray(size);
+    type->makeArray(arraySize);
 
     return new TField(type, identifier, loc);
 }