Fixed|Renderer: Reset GL state to defaults after rendering a model

Todo: GL state should be managed via GLState to avoid this kind of problems.
skyjake · Dec 21, 2015 · 4b54789 · 4b54789
1 parent c05904d
commit 4b54789
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/doomsday/apps/client/src/render/modelrenderer.cpp b/doomsday/apps/client/src/render/modelrenderer.cpp
@@ -746,8 +746,13 @@ void ModelRenderer::render(vissprite_t const &spr)
     d->draw(p);
     GLState::pop();
 
-    /// @todo Something is interfering with the cull setting elsewhere (remove this).
-    GLState::current().setCull(gl::Back).apply();
+    /// @todo Something is interfering with the GL state elsewhere (remove this).
+    GLState::current()
+            .setCull(gl::Back)
+            .setDepthTest(true)
+            .setDepthFunc(gl::Less)
+            .setDepthWrite(true)
+            .apply();
 }
 
 void ModelRenderer::render(vispsprite_t const &pspr)
@@ -771,8 +776,13 @@ void ModelRenderer::render(vispsprite_t const &pspr)
     d->draw(p);
     GLState::pop();
 
-    /// @todo Something is interfering with the cull setting elsewhere (remove this).
-    GLState::current().setCull(gl::Back).apply();
+    /// @todo Something is interfering with the GL state elsewhere (remove this).
+    GLState::current()
+            .setCull(gl::Back)
+            .setDepthTest(true)
+            .setDepthFunc(gl::Less)
+            .setDepthWrite(true)
+            .apply();
 }
 
 String ModelRenderer::shaderName(GLProgram const &program) const