defensive countermeasure to bug in Renoise

src/detail/vst3/os/osutil.h

@@ -7,12 +7,42 @@
 
 #include <atomic>
 #include <string>
+#include <functional>
 #define FMT_HEADER_ONLY 1
 #include "fmt/format.h"
 #include "fmt/ranges.h"
 
 namespace os
 {
+
+class State
+{
+ public:
+  State(std::function<void()> on, std::function<void()> off) : _on(on), _off(off)
+  {
+  }
+  void on()
+  {
+    if (!_state)
+    {
+      _on();
+    }
+    _state = true;
+  }
+  void off()
+  {
+    if (_state)
+    {
+      _off();
+    }
+    _state = false;
+  }
+
+ private:
+  bool _state = false;
+  std::function<void()> _on, _off;
+};
+
 class IPlugObject
 {
  public:
@@ -90,4 +120,4 @@ class fixedqueue
   std::atomic_uint32_t _head = 0u;
   std::atomic_uint32_t _tail = 0u;
 };
-};  // namespace util
+};  // namespace util

src/wrapasvst3.cpp

@@ -60,6 +60,7 @@ tresult PLUGIN_API ClapAsVst3::terminate()
 {
   if (_plugin)
   {
+    _os_attached.off();  // ensure we are detached (countermeasure to Bug in Renoise)
     _plugin->terminate();
     _plugin.reset();
   }
@@ -87,11 +88,12 @@ tresult PLUGIN_API ClapAsVst3::setActive(TBool state)
         _expressionmap & clap_supported_note_expressions::AS_VST3_NOTE_EXPRESSION_TUNING);
     updateAudioBusses();
 
-    os::attach(this);
+    _os_attached.on();
   }
   if (!state)
   {
-    os::detach(this);
+    _os_attached.off();
+
     if (_active)
     {
       _plugin->deactivate();

src/wrapasvst3.h

@@ -108,6 +108,7 @@ class ClapAsVst3 : public Steinberg::Vst::SingleComponentEffect,
     , _library(lib)
     , _libraryIndex(number)
     , _creationcontext(context)
+    , _os_attached([this] { os::attach(this); }, [this] { os::detach(this); })
   {
   }
 
@@ -253,6 +254,7 @@ class ClapAsVst3 : public Steinberg::Vst::SingleComponentEffect,
 
   // plugin state
   bool _active = false;
+  os::State _os_attached;
   bool _processing = false;
   std::mutex _processingLock;
   std::atomic_bool _requestedFlush = false;