comparison audioio/AudioCallbackPlaySource.cpp @ 204:5ee9e6bc21eb

* Use a mutex when setting auditioning plugin; lifecycle management too fragile otherwise

 {
     RealTimePluginInstance *plugin = dynamic_cast<RealTimePluginInstance *>(a);
     if (a && !plugin) {
         std::cerr << "WARNING: AudioCallbackPlaySource::setAuditioningEffect: auditionable object " << a << " is not a real-time plugin instance" << std::endl;
     }
-    RealTimePluginInstance *formerPlugin = m_auditioningPlugin;
+
+    m_mutex.lock();
     m_auditioningPlugin = plugin;
     m_auditioningPluginBypassed = false;
-    if (formerPlugin) m_pluginScavenger.claim(formerPlugin);
+    m_mutex.unlock();
 }
 
 void
 AudioCallbackPlaySource::setSoloModelSet(std::set<Model *> s)
 {

 AudioCallbackPlaySource::applyAuditioningEffect(size_t count, float **buffers)
 {
     if (m_auditioningPluginBypassed) return;
     RealTimePluginInstance *plugin = m_auditioningPlugin;
     if (!plugin) return;
-
+    
     if (plugin->getAudioInputCount() != getTargetChannelCount()) {
 //        std::cerr << "plugin input count " << plugin->getAudioInputCount() 
 //                  << " != our channel count " << getTargetChannelCount()
 //                  << std::endl;
         return;