fix: add read/write lock to property load/save

ZL-Audio · Sep 28, 2023 · 19b2b21 · 19b2b21
1 parent 39392e0
commit 19b2b21
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/Source/State/property.cpp b/Source/State/property.cpp
@@ -22,13 +22,15 @@ namespace zlstate {
     }
 
     void Property::loadAPVTS(juce::AudioProcessorValueTreeState &apvts) {
+        const juce::ScopedReadLock myScopedLock (readWriteLock);
         auto file = uiFile->getFile();
         if (auto xml = juce::XmlDocument::parse (file)) {
             apvts.replaceState(juce::ValueTree::fromXml(*xml));
         }
     }
 
     void Property::saveAPVTS(juce::AudioProcessorValueTreeState &apvts) {
+        const juce::ScopedWriteLock myScopedLock (readWriteLock);
         auto file = uiFile->getFile();
         if (auto xml = apvts.copyState().createXml()) {
             xml->writeTo(file);

diff --git a/Source/State/property.h b/Source/State/property.h
@@ -28,6 +28,7 @@ namespace zlstate {
 
     private:
         std::unique_ptr<juce::PropertiesFile> uiFile;
+        juce::ReadWriteLock readWriteLock;
 
         inline auto static const path = juce::File::getSpecialLocation(juce::File::userApplicationDataDirectory).getFullPathName();
         inline auto static const uiPath = path + "/Audio/Presets/" + JucePlugin_Manufacturer + "/" + JucePlugin_Name + "/ui.xml";