commit 8b5e8fdb3c353d46c81243b31889856181ff66ae
author Olli Etuaho <oetuaho@nvidia.com> Fri Dec 15 14:59:15 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Wed Jan 03 18:20:55 2018 +0000
tree 05ed91646e38bf161099dabee12a95e3dee1e65f
parent 93b059db4a5ab007e5239dff61eae3656e8da286

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>

src/compiler/translator/Symbol.cpp

diff --git a/src/compiler/translator/Symbol.cpp b/src/compiler/translator/Symbol.cpp
index f85e7eb..c1009cb 100644
--- a/src/compiler/translator/Symbol.cpp
+++ b/src/compiler/translator/Symbol.cpp
@@ -36,6 +36,7 @@
     ASSERT(mSymbolType == SymbolType::BuiltIn || mExtension == TExtension::UNDEFINED);
     ASSERT(mName != nullptr || mSymbolType == SymbolType::AngleInternal ||
            mSymbolType == SymbolType::NotResolved || mSymbolType == SymbolType::Empty);
+    ASSERT(mName == nullptr || *mName != "");
 }
 
 const TString &TSymbol::name() const