🐛 Wipe cached LoaderData when stores are changed

refs #35

Author: fredemmott

src/windows/WindowsPlatform.cpp

@@ -208,11 +208,16 @@ void WindowsPlatform::MainLoop(const std::function<void()>& drawFrame) {
 
   const wil::unique_event changeEvent(
     CreateEventW(nullptr, FALSE, TRUE, nullptr));
-  const auto changeSubscriptions
-    = APILayerStore::Get()
-    | std::views::transform([e = changeEvent.get()](const auto store) {
-        return store->OnChange(std::bind_front(&SetEvent, e));
-      })
+  const auto changeSubscriptions = APILayerStore::Get()
+    | std::views::transform([e = changeEvent.get(), this](const auto store) {
+                                     return store->OnChange([e, this] {
+                                       {
+                                         const std::unique_lock lock(mMutex);
+                                         mLoaderData.reset();
+                                       }
+                                       SetEvent(e);
+                                     });
+                                   })
     | std::ranges::to<std::vector>();
 
   while (true) {
@@ -229,9 +234,12 @@ void WindowsPlatform::MainLoop(const std::function<void()>& drawFrame) {
 
     ResetEvent(changeEvent.get());
 
-    this->BeforeFrame();
-    drawFrame();
-    this->AfterFrame();
+    {
+      const std::unique_lock lock(mMutex);
+      this->BeforeFrame();
+      drawFrame();
+      this->AfterFrame();
+    }
 
     {
       const auto e = changeEvent.get();

src/windows/WindowsPlatform.hpp

@@ -5,6 +5,8 @@
 #include <wil/com.h>
 #include <wil/resource.h>
 
+#include <mutex>
+
 #include <d3d11.h>
 #include <dxgi1_2.h>
 
@@ -58,6 +60,7 @@ class WindowsPlatform final : public Platform {
     Config::MINIMUM_WINDOW_WIDTH,
     Config::MINIMUM_WINDOW_HEIGHT,
   };
+  std::mutex mMutex;
   std::optional<std::expected<LoaderData, LoaderData::Error>> mLoaderData;
 
   HWND CreateAppWindow();