texture_manager: use a mutex for ref counted use

JonathSpirit · Dec 23, 2024 · 13686bc · 13686bc
1 parent 0ca9a95
commit 13686bc
Showing 1 changed file with 21 additions and 2 deletions.
diff --git a/sources/manager/texture_manager.cpp b/sources/manager/texture_manager.cpp
@@ -21,14 +21,28 @@
 namespace fge::texture
 {
 
+namespace
+{
+
+std::mutex gSDLImageHandlerMutex;
+unsigned int gSDLImageHandlerCounter = 0;
+
+} // namespace
+
 bool TextureManager::initialize()
 {
     if (this->isInitialized())
     {
         return true;
     }
 
-    IMG_Init(IMG_INIT_JPG | IMG_INIT_PNG | IMG_INIT_TIF | IMG_INIT_WEBP | IMG_INIT_JXL | IMG_INIT_AVIF);
+    gSDLImageHandlerMutex.lock();
+    if (gSDLImageHandlerCounter == 0)
+    {
+        IMG_Init(IMG_INIT_JPG | IMG_INIT_PNG | IMG_INIT_TIF | IMG_INIT_WEBP | IMG_INIT_JXL | IMG_INIT_AVIF);
+    }
+    ++gSDLImageHandlerCounter;
+    gSDLImageHandlerMutex.unlock();
 
     fge::Surface badSurface;
 
@@ -66,7 +80,12 @@ bool TextureManager::initialize()
 void TextureManager::uninitialize()
 {
     BaseManager::uninitialize();
-    IMG_Quit();
+    gSDLImageHandlerMutex.lock();
+    if (--gSDLImageHandlerCounter == 0)
+    {
+        IMG_Quit();
+    }
+    gSDLImageHandlerMutex.unlock();
 }
 
 bool TextureManager::loadFromSurface(std::string_view name, fge::Surface const& surface)