Tiny mistake in previous commit.

This commit is contained in:
JoelKatz
2011-11-11 16:38:49 -08:00
parent 3452211310
commit 3f8f7789b3

View File

@@ -6,15 +6,15 @@
// This is a returnable lock holder. // This is a returnable lock holder.
// I don't know why Boost doesn't provide a good way to do this. // I don't know why Boost doesn't provide a good way to do this.
class ScopedLock class ScopedLock
{ {
private: private:
boost::mutex *mMutex; // parent object has greater scope, so guaranteed valid boost::mutex *mMutex; // parent object has greater scope, so guaranteed valid
mutable bool mValid; mutable bool mValid;
ScopedLock(); // no implementation ScopedLock(); // no implementation
public: public:
ScopedLock(boost::mutex &mutex) : mMutex(&mutex), mValid(true) ScopedLock(boost::mutex &mutex) : mMutex(&mutex), mValid(true)
{ {
mMutex->lock(); mMutex->lock();
@@ -42,11 +42,10 @@ public:
mMutex=sl.mMutex; mMutex=sl.mMutex;
if(sl.mValid) if(sl.mValid)
{ {
if(mValid) mMutex->unlock();
mValid=true; mValid=true;
sl.mValid=false; sl.mValid=false;
} }
} }
}; };
#endif #endif