commit 5ed23982b0ce180cf0d33d67b7b066fb9a1a0f2e
author Jamie Madill <jmadill@chromium.org> Fri Apr 22 15:08:57 2016 -0400
committer Commit Bot <commit-bot@chromium.org> Tue Apr 26 17:07:43 2016 +0000
tree e355f38468df85dc41cd9df14ef162fb497e0cbe
parent d4f4c11b92cb03d121bda62fa75f605d8b190940

Fix allocation in ValidateOutputs.

We should not be using the pool allocator for destructable objects.

BUG=None

Change-Id: I89236b28f04bd9b7095056edbda4172dbbfe9586
Reviewed-on: https://chromium-review.googlesource.com/340362
Reviewed-by: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>

src/compiler/translator/ValidateOutputs.cpp

diff --git a/src/compiler/translator/ValidateOutputs.cpp b/src/compiler/translator/ValidateOutputs.cpp
index cd37aea..29ca68d 100644
--- a/src/compiler/translator/ValidateOutputs.cpp
+++ b/src/compiler/translator/ValidateOutputs.cpp
@@ -34,10 +34,10 @@
     TString name = symbol->getSymbol();
     TQualifier qualifier = symbol->getQualifier();
 
-    if (mVisitedSymbols.count(name) == 1)
+    if (mVisitedSymbols.count(name.c_str()) == 1)
         return;
 
-    mVisitedSymbols.insert(name);
+    mVisitedSymbols.insert(name.c_str());
 
     if (qualifier == EvqFragmentOut)
     {

src/compiler/translator/ValidateOutputs.h

diff --git a/src/compiler/translator/ValidateOutputs.h b/src/compiler/translator/ValidateOutputs.h
index 06f6399..0122ca2 100644
--- a/src/compiler/translator/ValidateOutputs.h
+++ b/src/compiler/translator/ValidateOutputs.h
@@ -30,7 +30,7 @@
     typedef std::vector<TIntermSymbol *> OutputVector;
     OutputVector mOutputs;
     OutputVector mUnspecifiedLocationOutputs;
-    std::set<TString> mVisitedSymbols;
+    std::set<std::string> mVisitedSymbols;
 };
 
 #endif // COMPILER_TRANSLATOR_VALIDATEOUTPUTS_H_