X-Git-Url: http://git.tdb.fi/?p=libs%2Fcore.git;a=blobdiff_plain;f=source%2Fcore%2Fmutex.h;h=6582074e182c988c3d813671b3cb92668c27ceeb;hp=050d00f808a6d42f29e8cbf14744e935bad604d4;hb=HEAD;hpb=967785734be5c3fc6f75da122c2d93ebbb338271

diff --git a/source/core/mutex.h b/source/core/mutex.h
index 050d00f..a48f58c 100644
--- a/source/core/mutex.h
+++ b/source/core/mutex.h
@@ -1,6 +1,8 @@
 #ifndef MSP_CORE_MUTEX_H_
 #define MSP_CORE_MUTEX_H_
 
+#include "mspcore_api.h"
+#include "noncopyable.h"
 #include "refptr.h"
 
 namespace Msp {
@@ -9,14 +11,14 @@ namespace Msp {
 A class for controlling mutually exclusive access to a resource.  Only one
 thread can hold a lock on the mutex at a time.
 */
-class Mutex
+class MSPCORE_API Mutex: private NonCopyable
 {
 	friend class Semaphore;
 
 private:
 	struct Private;
 
-	Private *priv;
+	Private *priv = nullptr;
 
 public:
 	Mutex();
@@ -35,21 +37,19 @@ public:
 };
 
 /**
-Locks the mutex for te lifetime of the object.
+Locks the mutex for the lifetime of the object.
 */
-class MutexLock
+class MSPCORE_API MutexLock
 {
 private:
 	Mutex &mutex;
 
+	MutexLock(const MutexLock &);
 public:
 	MutexLock(Mutex &m, bool l = true): mutex(m) { if(l) mutex.lock(); }
 	~MutexLock() { mutex.unlock(); }
 
 	void lock() { mutex.lock(); }
-private:
-	MutexLock(const MutexLock &);
-	MutexLock &operator=(const MutexLock &);
 };
 
 /**
@@ -59,41 +59,17 @@ exists, the mutex will stay locked.
 template<typename T>
 class MutexPtr
 {
-public:
-	MutexPtr(T *d, Mutex &m): mutex(new MutexLock(m)), data(d) { }
-
-	T &operator*() const { return *data; }
-	T *operator->() const { return data; }
-	void clear() { mutex=0; data = 0; }
 private:
 	RefPtr<MutexLock> mutex;
 	T *data;
-};
 
-/*template<typename T>
-class MutexPtr: public RefCount
-{
 public:
-	MutexPtr(T *d, Mutex &m): mutex(m), data(d) { mutex.lock(); }
-	MutexPtr(const MutexPtr<T> &p): RefCount(p), mutex(p.mutex), data(p.data) { }
+	MutexPtr(T *d, Mutex &m): mutex(new MutexLock(m)), data(d) { }
+
 	T &operator*() const { return *data; }
 	T *operator->() const { return data; }
-	void clear() { decref(); data = 0; }
-	~MutexPtr() { decref(); }
-protected:
-	Mutex &mutex;
-	T     *data;
-
-	bool decref()
-	{
-		if(!RefCount::decref())
-		{
-			mutex.unlock();
-			return false;
-		}
-		return true;
-	}
-};*/
+	void clear() { mutex = nullptr; data = nullptr; }
+};
 
 }