commit a3fe2b4e1fce3d0637e2626c6154d19707313a85
author Jamie Madill <jmadill@chromium.org> Fri Jul 18 10:33:13 2014 -0400
committer Jamie Madill <jmadill@chromium.org> Thu Jul 31 20:38:06 2014 +0000
tree 14370b8312ede395402495ab6a5766b451b22a60
parent f8bdfebc5eefb90dc973366e711f1295a6026b77

Update non-default constructors in shadervars.h.

Most of these constructors are unused. Remove the unused ones,
and change the usage of existing constructors to be consistent.

BUG=angle:466,697

Change-Id: I455dd314036e1dfcb8c4690bab577b81dd0e060c
Reviewed-on: https://chromium-review.googlesource.com/208355
Tested-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Nicolas Capens <capn@chromium.org>

src/compiler/translator/OutputHLSL.cpp

diff --git a/src/compiler/translator/OutputHLSL.cpp b/src/compiler/translator/OutputHLSL.cpp
index 1bf1181..ed082c9 100644
--- a/src/compiler/translator/OutputHLSL.cpp
+++ b/src/compiler/translator/OutputHLSL.cpp
@@ -29,6 +29,18 @@
 namespace sh
 {
 
+static sh::Attribute MakeAttributeFromType(const TType &type, const TString &name)
+{
+    sh::Attribute attributeVar;
+    attributeVar.type = GLVariableType(type);
+    attributeVar.precision = GLVariablePrecision(type);
+    attributeVar.name = name.c_str();
+    attributeVar.arraySize = static_cast<unsigned int>(type.getArraySize());
+    attributeVar.location = type.getLayoutQualifier().location;
+
+    return attributeVar;
+}
+
 TString OutputHLSL::TextureFunction::name() const
 {
     TString name = "gl_texture";
@@ -335,8 +347,7 @@
 
         attributes += "static " + TypeString(type) + " " + Decorate(name) + ArrayString(type) + " = " + initializer(type) + ";\n";
 
-        sh::Attribute attributeVar(GLVariableType(type), GLVariablePrecision(type), name.c_str(),
-                               (unsigned int)type.getArraySize(), type.getLayoutQualifier().location);
+        sh::Attribute attributeVar = MakeAttributeFromType(type, name);
         mActiveAttributes.push_back(attributeVar);
     }
 
@@ -370,13 +381,11 @@
             {
                 const TString &variableName = outputVariableIt->first;
                 const TType &variableType = outputVariableIt->second->getType();
-                const TLayoutQualifier &layoutQualifier = variableType.getLayoutQualifier();
 
                 out << "static " + TypeString(variableType) + " out_" + variableName + ArrayString(variableType) +
                        " = " + initializer(variableType) + ";\n";
 
-                sh::Attribute outputVar(GLVariableType(variableType), GLVariablePrecision(variableType), variableName.c_str(),
-                                    (unsigned int)variableType.getArraySize(), layoutQualifier.location);
+                sh::Attribute outputVar = MakeAttributeFromType(variableType, variableName);
                 mActiveOutputVariables.push_back(outputVar);
             }
         }