commit 722bfb51d59188ee086839b69e00296b2d5d2da3
author Olli Etuaho <oetuaho@nvidia.com> Thu Oct 26 17:00:11 2017 +0300
committer Commit Bot <commit-bot@chromium.org> Tue Oct 31 07:31:35 2017 +0000
tree 66cc46bf773ab18e81bf568248185880af4c6147
parent 9088557fe47e0ce1487b248b60acbc740acd7801

Fix detecting duplicate field names in structures

Previously field names that were listed in the first declarator list
inside a struct declaration were not checked against each other.

BUG=angleproject:2204
TEST=angle_unittests

Change-Id: Ibf821d45556f6dfe0223dae673644f6795daf4cb
Reviewed-on: https://chromium-review.googlesource.com/739825
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.h

diff --git a/src/compiler/translator/ParseContext.h b/src/compiler/translator/ParseContext.h
index f423b83..1f7f5b9 100644
--- a/src/compiler/translator/ParseContext.h
+++ b/src/compiler/translator/ParseContext.h
@@ -306,6 +306,11 @@
                                        unsigned int arraySize,
                                        const TSourceLoc &arraySizeLoc);
 
+    void checkDoesNotHaveDuplicateFieldName(const TFieldList::const_iterator begin,
+                                            const TFieldList::const_iterator end,
+                                            const TString &name,
+                                            const TSourceLoc &location);
+    TFieldList *addStructFieldList(TFieldList *fields, const TSourceLoc &location);
     TFieldList *combineStructFieldLists(TFieldList *processedFields,
                                         const TFieldList *newlyAddedFields,
                                         const TSourceLoc &location);