Remove a call to GetModel (which involves a mutex) for each frame of …

…each effect being rendered.
xLightsSequencer · Dec 29, 2023 · 9f8bc53 · 9f8bc53
1 parent f25642b
commit 9f8bc53
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/macOS b/macOS
diff --git a/xLights/RenderBuffer.cpp b/xLights/RenderBuffer.cpp
@@ -1529,14 +1529,16 @@ void RenderBuffer::SetState(int period, bool ResetState, const std::string& mode
         needToInit = true;
     }
     curPeriod = period;
-    cur_model = model_name;
     curPeriod = period;
     palette.UpdateForProgress(GetEffectTimeIntervalPosition());
-    dmx_buffer = false;
-    Model* m = GetModel();
-    if (m != nullptr) {
-        if (m->GetDisplayAs().rfind("Dmx", 0) == 0) {
-            dmx_buffer = true;
+    if (cur_model != model_name) {
+        cur_model = model_name;
+        dmx_buffer = false;
+        Model* m = GetModel();
+        if (m != nullptr) {
+            if (m->GetDisplayAs().rfind("Dmx", 0) == 0) {
+                dmx_buffer = true;
+            }
         }
     }
 }