Commit 10567260 by Jamie Madill

Fix build with Clang.

Clang's validation is a bit more thorough than GCC's or MSVC's and picked up a few new errors and warnings. We missed a few default cases in switches and used an improper type for default arguments. BUG=angle:568 Change-Id: I34239c6c5c785ad1e62917fc37c3243f4c5b5f1b Reviewed-on: https://chromium-review.googlesource.com/195472Tested-by: 's avatarJamie Madill <jmadill@chromium.org> Reviewed-by: 's avatarShannon Woods <shannonwoods@chromium.org> Reviewed-by: 's avatarGeoff Lang <geofflang@chromium.org>
parent 4f267868
......@@ -4192,6 +4192,8 @@ bool OutputHLSL::isVaryingOut(TQualifier qualifier)
case EvqCentroidOut:
case EvqVertexOut:
return true;
default: break;
}
return false;
......@@ -4208,6 +4210,8 @@ bool OutputHLSL::isVaryingIn(TQualifier qualifier)
case EvqCentroidIn:
case EvqFragmentIn:
return true;
default: break;
}
return false;
......
......@@ -638,6 +638,8 @@ bool TParseContext::structQualifierErrorCheck(const TSourceLoc& line, const TPub
error(line, "cannot be used with a structure", getQualifierString(pType.qualifier));
return true;
}
default: break;
}
if (pType.qualifier != EvqUniform && samplerErrorCheck(line, pType, "samplers must be uniform"))
......
......@@ -360,7 +360,7 @@ public:
insert(level, *function);
}
TSymbol *find(const TString &name, int shaderVersion, bool *builtIn = false, bool *sameScope = false);
TSymbol *find(const TString &name, int shaderVersion, bool *builtIn = NULL, bool *sameScope = NULL);
TSymbol *findBuiltIn(const TString &name, int shaderVersion);
TSymbolTableLevel *getOuterLevel() {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment