GPU: Add more safeguard for BGL calls

This makes sure no BGL call before window drawing locks the GPUState.

1 files changed, 16 insertions, 11 deletions

diff --git a/source/blender/gpu/intern/gpu_state.cc b/source/blender/gpu/intern/gpu_state.cc
index 4bdd49cb210..d0048ab9b87 100644
--- a/source/blender/gpu/intern/gpu_state.cc
+++ b/source/blender/gpu/intern/gpu_state.cc
@@ -330,23 +330,28 @@ void GPU_apply_state(void)
 void GPU_bgl_start(void)
 {
   Context *ctx = Context::get();
-  if (ctx && ctx->state_manager) {
-    StateManager &state_manager = *(Context::get()->state_manager);
-    if (state_manager.use_bgl == false) {
-      /* Expected by many addons (see T80169, T81289).
-       * This will reset the blend function. */
-      GPU_blend(GPU_BLEND_NONE);
-      state_manager.apply_state();
-      state_manager.use_bgl = true;
-    }
+  if (!(ctx && ctx->state_manager)) {
+    return;
+  }
+  StateManager &state_manager = *(Context::get()->state_manager);
+  if (state_manager.use_bgl == false) {
+    /* Expected by many addons (see T80169, T81289).
+     * This will reset the blend function. */
+    GPU_blend(GPU_BLEND_NONE);
+    state_manager.apply_state();
+    state_manager.use_bgl = true;
   }
 }
 
+/* Just turn off the bgl safeguard system. Can be called even without GPU_bgl_start. */
 void GPU_bgl_end(void)
 {
   Context *ctx = Context::get();
-  if (ctx && ctx->state_manager) {
-    StateManager &state_manager = *ctx->state_manager;
+  if (!(ctx && ctx->state_manager)) {
+    return;
+  }
+  StateManager &state_manager = *ctx->state_manager;
+  if (state_manager.use_bgl == true) {
     state_manager.use_bgl = false;
     /* Resync state tracking. */
     state_manager.force_state();