X-Git-Url: http://git.tdb.fi/?p=libs%2Fcore.git;a=blobdiff_plain;f=source%2Fcore%2Fmutex.h;h=6582074e182c988c3d813671b3cb92668c27ceeb;hp=50a64e55aaa0a87a1a6d114960ed5cd3416e5527;hb=41363aed34382386f915f17c1a961750b4fdcb14;hpb=cfc8e0b7b15ea505bd6a6a9599cbc5ce1e316963

diff --git a/source/core/mutex.h b/source/core/mutex.h
index 50a64e5..6582074 100644
--- a/source/core/mutex.h
+++ b/source/core/mutex.h
@@ -1,57 +1,54 @@
-/* $Id$
-
-This file is part of libmspcore
-Copyright © 2006 Mikko Rasa, Mikkosoft Productions
-Distributed under the LGPL
-*/
-
 #ifndef MSP_CORE_MUTEX_H_
 #define MSP_CORE_MUTEX_H_
 
+#include "noncopyable.h"
 #include "refptr.h"
-#include "types.h"
 
 namespace Msp {
 
-class Mutex
+/**
+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: private NonCopyable
 {
 	friend class Semaphore;
 
 private:
-	MutexHandle mutex;
+	struct Private;
+
+	Private *priv = nullptr;
 
 public:
-#ifndef WIN32
-	Mutex()       { pthread_mutex_init(&mutex, 0); }
-	int lock()    { return pthread_mutex_lock(&mutex); }
-	int trylock() { return pthread_mutex_trylock(&mutex); }
-	int unlock()  { return pthread_mutex_unlock(&mutex); }
-	~Mutex()      { pthread_mutex_destroy(&mutex); }
-#else
-	Mutex()       { mutex=CreateMutex(0, false, 0); }
-	int lock()    { return WaitForSingleObject(mutex, INFINITE)==WAIT_OBJECT_0; }
-	int trylock() { return WaitForSingleObject(mutex, 0)==WAIT_OBJECT_0; }
-	int unlock()  { return !ReleaseMutex(mutex); }
-	~Mutex()      { CloseHandle(mutex); }
-#endif
+	Mutex();
+	~Mutex();
+
+	/** Locks the mutex.  If the mutex is already locked, waits until it becomes
+	available. */
+	void lock();
+
+	/** Attempts to lock the mutex.  Returns true if the lock was acquired,
+	otherwise returns false. */
+	bool trylock();
+
+	/** Unlocks the mutex. */
+	void unlock();
 };
 
 /**
-Locks the mutex for te lifetime of the object.
+Locks the mutex for the lifetime of the object.
 */
 class MutexLock
 {
 private:
 	Mutex &mutex;
 
+	MutexLock(const MutexLock &);
 public:
-	MutexLock(Mutex &m, bool l=true): mutex(m) { if(l) mutex.lock(); }
+	MutexLock(Mutex &m, bool l = true): mutex(m) { if(l) mutex.lock(); }
 	~MutexLock() { mutex.unlock(); }
 
-	int lock() { return mutex.lock(); }
-private:
-	MutexLock(const MutexLock &);
-	MutexLock &operator=(const MutexLock &);
+	void lock() { mutex.lock(); }
 };
 
 /**
@@ -61,41 +58,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; }
+};
 
 }