Replace remaining usage of TName with TSymbol
TName used to contain just a subset of the information stored in
TSymbol. It makes more sense to use TSymbol directly instead of
converting it to TName.
This also improves type safety a bit by making some functions only
take in TVariable or TFunction instead of the more generic TName.
BUG=angleproject:2267
TEST=angle_unittests
Change-Id: Icb46923c25d33ebbbbc06ddc487da25957dda771
Reviewed-on: https://chromium-review.googlesource.com/829143
Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
diff --git a/src/compiler/translator/ValidateOutputs.cpp b/src/compiler/translator/ValidateOutputs.cpp
index 2f76d6f..107c845 100644
--- a/src/compiler/translator/ValidateOutputs.cpp
+++ b/src/compiler/translator/ValidateOutputs.cpp
@@ -20,9 +20,10 @@
namespace
{
+
void error(const TIntermSymbol &symbol, const char *reason, TDiagnostics *diagnostics)
{
- diagnostics->error(symbol.getLine(), reason, symbol.getSymbol().c_str());
+ diagnostics->error(symbol.getLine(), reason, symbol.getName().c_str());
}
class ValidateOutputsTraverser : public TIntermTraverser
@@ -112,7 +113,7 @@
{
std::stringstream strstr;
strstr << "conflicting output locations with previously defined output '"
- << validOutputs[offsetLocation]->getSymbol() << "'";
+ << validOutputs[offsetLocation]->getName() << "'";
error(*symbol, strstr.str().c_str(), diagnostics);
}
else