[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.

1 files changed, 4 insertions, 2 deletions

diff --git a/test/gl/object.cpp b/test/gl/object.cpp
index 5e7a158939..1506c33956 100644
--- a/test/gl/object.cpp
+++ b/test/gl/object.cpp
@@ -18,11 +18,13 @@ static bool setFlag = false;
 
 struct MockGLObject {
     using Type = bool;
-    static const Type Default = false;
+    static const Type Default;
     static Type Get() { getFlag = true; return true; }
     static void Set(const Type&) { setFlag = true; }
 };
 
+const bool MockGLObject::Default = false;
+
 TEST(GLObject, Preserve) {
     getFlag = false;
     setFlag = false;
@@ -61,7 +63,7 @@ TEST(GLObject, Value) {
 
     object->reset();
     EXPECT_EQ(object->getCurrent(), false);
-    EXPECT_TRUE(object->getDirty());
+    EXPECT_FALSE(object->getDirty());
     EXPECT_TRUE(setFlag);
 }