BeeEngine/glslang-zig
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

WIP: apply unused variable

Browse source 
I happened upon numArgs while hunting for unused variables. I suspect
the intent was to apply it as shown in this patch. However, I am not a
compiler dude. Someone more appropriate should grok this change.
This commit is contained in:
Jeremy Hayes 2016-10-20 16:12:36 -06:00
parent 5d45eadedc
commit c8e60e28b7
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
glslang/MachineIndependent/ParseHelper.cpp
View file

@ -1216,7 +1216,7 @@ void TParseContext::computeBuiltinPrecisions(TIntermTyped& node, const TFunction
function.getType().getQualifier().precision; function.getType().getQualifier().precision;
} else if (TIntermAggregate* agg = node.getAsAggregate()) { } else if (TIntermAggregate* agg = node.getAsAggregate()) {
TIntermSequence& sequence = agg->getSequence(); TIntermSequence& sequence = agg->getSequence();
int numArgs = (int)sequence.size(); unsigned int numArgs = (unsigned int)sequence.size();
switch (agg->getOp()) { switch (agg->getOp()) {
case EOpBitfieldExtract: case EOpBitfieldExtract:
numArgs = 1; numArgs = 1;
@ -1233,7 +1233,7 @@ void TParseContext::computeBuiltinPrecisions(TIntermTyped& node, const TFunction
break; break;
} }
// find the maximum precision from the arguments and parameters // find the maximum precision from the arguments and parameters
for (unsigned int arg = 0; arg < sequence.size(); ++arg) { for (unsigned int arg = 0; arg < numArgs; ++arg) {
operationPrecision = std::max(operationPrecision, sequence[arg]->getAsTyped()->getQualifier().precision); operationPrecision = std::max(operationPrecision, sequence[arg]->getAsTyped()->getQualifier().precision);
operationPrecision = std::max(operationPrecision, function[arg].type->getQualifier().precision); operationPrecision = std::max(operationPrecision, function[arg].type->getQualifier().precision);
} }