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/340362Reviewed-by: Olli Etuaho <oetuaho@nvidia.com> Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>

Fix allocation in ValidateOutputs.
5ed23982 · Jamie Madill · Commit Bot · d4f4c11b · 5ed23982 · 5ed23982
Commit 5ed23982 authored Apr 22, 2016 by Jamie Madill Committed by Commit Bot Apr 26, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

ValidateOutputs.cpp src/compiler/translator/ValidateOutputs.cpp +2 -2

ValidateOutputs.h src/compiler/translator/ValidateOutputs.h +1 -1

No files found.
--- a/src/compiler/translator/ValidateOutputs.cpp
+++ b/src/compiler/translator/ValidateOutputs.cpp
@@ -34,10 +34,10 @@ void ValidateOutputs::visitSymbol(TIntermSymbol *symbol)
    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)
    {

--- a/src/compiler/translator/ValidateOutputs.h
+++ b/src/compiler/translator/ValidateOutputs.h
@@ -30,7 +30,7 @@ class ValidateOutputs : public TIntermTraverser
    typedef std::vector<TIntermSymbol *> OutputVector;
    OutputVector mOutputs;
    OutputVector mUnspecifiedLocationOutputs;
-    std::set<TString> mVisitedSymbols;
+    std::set<std::string> mVisitedSymbols;
 };

 #endif // COMPILER_TRANSLATOR_VALIDATEOUTPUTS_H_