[core] don't force GL state to be set when calling .reset()

We are using Value::reset() to change a piece of GL state to its default value. However, the current implementation always executes the GL call, even if our state tracking system knows that it's already at that value. The new implementation of Value::reset() now respects that, resulting in a lot fewer GL calls.

2 files changed, 2 insertions, 3 deletions

diff --git a/src/mbgl/gl/gl_config.hpp b/src/mbgl/gl/gl_config.hpp
index 09080ca365..9d7dfb3b6c 100644
--- a/src/mbgl/gl/gl_config.hpp
+++ b/src/mbgl/gl/gl_config.hpp
@@ -21,9 +21,7 @@ public:
     }
 
     void reset() {
-        dirty = true;
-        current = T::Default;
-        T::Set(current);
+        *this = T::Default;
     }
 
     void setDirty() {
diff --git a/src/mbgl/renderer/painter.cpp b/src/mbgl/renderer/painter.cpp
index e5c88de635..ba68b5a24d 100644
--- a/src/mbgl/renderer/painter.cpp
+++ b/src/mbgl/renderer/painter.cpp
@@ -83,6 +83,7 @@ Painter::Painter(const TransformState& state_,
     overdrawShader.circle = std::make_unique<CircleShader>(store, Shader::Overdraw);
 
     // Reset GL values
+    config.setDirty();
     config.reset();
 }