Use pthread_once() to guard the singleton initialization.

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@34384 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/kits/storage/disk_device/DiskSystemAddOnManager.cpp

@@ -1,5 +1,5 @@
 /*
- * Copyright 2007, Ingo Weinhold, [email protected].
+ * Copyright 2007-2009, Ingo Weinhold, [email protected].
  * Distributed under the terms of the MIT License.
  */
 
@@ -17,7 +17,6 @@
 #include <image.h>
 #include <Path.h>
 
-#include <AppMisc.h>
 #include <AutoDeleter.h>
 #include <AutoLocker.h>
 
@@ -32,7 +31,7 @@
 using std::nothrow;
 
 
-// sManager
+static pthread_once_t sManagerInitOnce = PTHREAD_ONCE_INIT;
 DiskSystemAddOnManager* DiskSystemAddOnManager::sManager = NULL;
 
 
@@ -78,22 +77,8 @@ struct DiskSystemAddOnManager::StringSet : std::set<std::string> {
 DiskSystemAddOnManager*
 DiskSystemAddOnManager::Default()
 {
-	if (!sManager) {
+	if (sManager == NULL)
-		DiskSystemAddOnManager* manager = new DiskSystemAddOnManager();
+		pthread_once(&sManagerInitOnce, &_InitSingleton);
-
-		BPrivate::gInitializationLock.Lock();
-
-		// set manager, if no one beat us to it
-		if (!sManager) {
-			sManager = manager;
-			manager = NULL;
-		}
-
-		BPrivate::gInitializationLock.Unlock();
-
-		// delete the object we created, if someone else was quicker
-		delete manager;
-	}
 
 	return sManager;
 }
@@ -238,6 +223,13 @@ DiskSystemAddOnManager::DiskSystemAddOnManager()
 }
 
 
+/*static*/ void
+DiskSystemAddOnManager::_InitSingleton()
+{
+	sManager = new DiskSystemAddOnManager();
+}
+
+
 // _AddOnAt
 DiskSystemAddOnManager::AddOn*
 DiskSystemAddOnManager::_AddOnAt(int32 index) const
@@ -369,4 +361,3 @@ DiskSystemAddOnManager::_LoadAddOns(StringSet& alreadyLoaded,
 
 	return B_OK;
 }
-

src/kits/storage/disk_device/DiskSystemAddOnManager.h

@@ -42,6 +42,8 @@ private:
 
 								DiskSystemAddOnManager();
 
+	static	void				_InitSingleton();
+
 			AddOn*				_AddOnAt(int32 index) const;
 			void				_PutAddOn(int32 index);