Merge pull request #1791 from plugdata-team/develop

Fix plugin shutdown bug

Author: timothyschoen

Source/Sidebar/DocumentationBrowser.h

@@ -99,6 +99,7 @@ class DocumentationBrowserUpdateThread : public Thread
 
     ~DocumentationBrowserUpdateThread()
     {
+        
         instance = nullptr;
         stopThread(-1);
     }
@@ -202,20 +203,33 @@ class DocumentationBrowserUpdateThread : public Thread
         }
     }
 
-    void run() override
-    {
-        try
-        {
-            fileTreeLock.enter();
-            fileTree = generateDirectoryValueTree(File(SettingsFile::getInstance()->getProperty<String>("browser_path")));
-            fileTreeLock.exit();
-            sendChangeMessage();
-        }
-        catch(...)
+        void run() override
         {
-            std::cerr << "Failed to update documentation browser" << std::endl;
+            try
+            {
+                int const maxRetries = 50;
+                int retries = 0;
+
+                while (retries < maxRetries) {
+                    if (threadShouldExit())
+                        break;
+                    if (fileTreeLock.tryEnter()) {
+                        fileTree = generateDirectoryValueTree(File(SettingsFile::getInstance()->getProperty<String>("browser_path")));
+                        fileTreeLock.exit();
+                        break;
+                    }
+                    retries++;
+                    Time::waitForMillisecondCounter(Time::getMillisecondCounter() + 100);
+                }
+                
+
+                sendChangeMessage();
+            }
+            catch(...)
+            {
+                std::cerr << "Failed to update documentation browser" << std::endl;
+            }
         }
-    }
 
     void filesystemChanged() override
     {