[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdeplasma-addons/applets/frame
From:       Aaron J. Seigo <aseigo () kde ! org>
Date:       2011-01-21 23:34:23
Message-ID: 20110121233423.11832AC8B8 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1216214 by aseigo:

work around non-thread safety of kexiv
BUG:263825


 M  +9 -0      imageloader.cpp  


--- trunk/KDE/kdeplasma-addons/applets/frame/imageloader.cpp #1216213:1216214
@@ -29,6 +29,15 @@
 ImageLoader::ImageLoader(const QString &path)
 {
     m_path = path;
+#ifdef HAVE_KEXIV2 
+    // prevets crashes due the xmp library not having a thread safe init
+    // may get fixed in future version of exiv2 according to devs
+    // FIXME: due to not knowing when it is safe to *uninitialize* exiv2 since others
+    // may be using it, this may ultimately result in a small one-time memory leak;
+    // either we need to know all users of KExiv2, or the thread safety issue needs
+    // to get fixed.
+    KExiv2Iface::KExiv2::initializeExiv2();
+#endif
 }
 
 QImage ImageLoader::correctRotation(const QImage& tempImage, const QString &path)
[prev in list] [next in list] [prev in thread] [next in thread]