Fix minor bug where we didn't clear dirtyUniforms if early-outing from UpdateUniform.

This commit is contained in:
Henrik Rydgård 2013-11-13 10:02:58 +01:00
parent 04b338f39e
commit 67ca4419fe

View file

@ -346,6 +346,7 @@ void LinkedShader::stop() {
void LinkedShader::UpdateUniforms(u32 vertType) { void LinkedShader::UpdateUniforms(u32 vertType) {
u32 dirty = dirtyUniforms & availableUniforms; u32 dirty = dirtyUniforms & availableUniforms;
dirtyUniforms = 0;
if (!dirty) if (!dirty)
return; return;
@ -441,7 +442,7 @@ void LinkedShader::UpdateUniforms(u32 vertType) {
bool allDirty = true; bool allDirty = true;
for (int i = 0; i < numBones; i++) { for (int i = 0; i < numBones; i++) {
if (dirtyUniforms & (DIRTY_BONEMATRIX0 << i)) { if (dirty & (DIRTY_BONEMATRIX0 << i)) {
ConvertMatrix4x3To4x4(gstate.boneMatrix + 12 * i, allBones + 16 * i); ConvertMatrix4x3To4x4(gstate.boneMatrix + 12 * i, allBones + 16 * i);
} else { } else {
allDirty = false; allDirty = false;
@ -453,7 +454,7 @@ void LinkedShader::UpdateUniforms(u32 vertType) {
} else { } else {
// Set them one by one. Could try to coalesce two in a row etc but too lazy. // Set them one by one. Could try to coalesce two in a row etc but too lazy.
for (int i = 0; i < numBones; i++) { for (int i = 0; i < numBones; i++) {
if (dirtyUniforms & (DIRTY_BONEMATRIX0 << i)) { if (dirty & (DIRTY_BONEMATRIX0 << i)) {
glUniformMatrix4fv(u_bone + i, 1, GL_FALSE, allBones + 16 * i); glUniformMatrix4fv(u_bone + i, 1, GL_FALSE, allBones + 16 * i);
} }
} }
@ -470,19 +471,19 @@ void LinkedShader::UpdateUniforms(u32 vertType) {
#endif #endif
// Lighting // Lighting
if (dirtyUniforms & DIRTY_AMBIENT) { if (dirty & DIRTY_AMBIENT) {
SetColorUniform3Alpha(u_ambient, gstate.ambientcolor, gstate.getAmbientA()); SetColorUniform3Alpha(u_ambient, gstate.ambientcolor, gstate.getAmbientA());
} }
if (dirtyUniforms & DIRTY_MATAMBIENTALPHA) { if (dirty & DIRTY_MATAMBIENTALPHA) {
SetColorUniform3Alpha(u_matambientalpha, gstate.materialambient, gstate.getMaterialAmbientA()); SetColorUniform3Alpha(u_matambientalpha, gstate.materialambient, gstate.getMaterialAmbientA());
} }
if (dirtyUniforms & DIRTY_MATDIFFUSE) { if (dirty & DIRTY_MATDIFFUSE) {
SetColorUniform3(u_matdiffuse, gstate.materialdiffuse); SetColorUniform3(u_matdiffuse, gstate.materialdiffuse);
} }
if (dirtyUniforms & DIRTY_MATEMISSIVE) { if (dirty & DIRTY_MATEMISSIVE) {
SetColorUniform3(u_matemissive, gstate.materialemissive); SetColorUniform3(u_matemissive, gstate.materialemissive);
} }
if (dirtyUniforms & DIRTY_MATSPECULAR) { if (dirty & DIRTY_MATSPECULAR) {
SetColorUniform3ExtraFloat(u_matspecular, gstate.materialspecular, getFloat24(gstate.materialspecularcoef)); SetColorUniform3ExtraFloat(u_matspecular, gstate.materialspecular, getFloat24(gstate.materialspecularcoef));
} }
@ -512,8 +513,6 @@ void LinkedShader::UpdateUniforms(u32 vertType) {
if (u_lightspecular[i] != -1) glUniform3fv(u_lightspecular[i], 1, gstate_c.lightColor[2][i]); if (u_lightspecular[i] != -1) glUniform3fv(u_lightspecular[i], 1, gstate_c.lightColor[2][i]);
} }
} }
dirtyUniforms = 0;
} }
ShaderManager::ShaderManager() : lastShader_(NULL), globalDirty_(0xFFFFFFFF), shaderSwitchDirty_(0) { ShaderManager::ShaderManager() : lastShader_(NULL), globalDirty_(0xFFFFFFFF), shaderSwitchDirty_(0) {