Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1451)

Unified Diff: native_client_sdk/src/libraries/sdk_util/auto_lock.h

Issue 18644009: [NaCl SDK] Upate atomic ops in nacl_io (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 'Move comment to correct location' Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: native_client_sdk/src/libraries/sdk_util/auto_lock.h
diff --git a/native_client_sdk/src/libraries/sdk_util/auto_lock.h b/native_client_sdk/src/libraries/sdk_util/auto_lock.h
index 88215d0cb31acb3c25c084be2692fe75870a3975..3dddb070ea48eaaa4f75e1095705eaff6bacf82a 100644
--- a/native_client_sdk/src/libraries/sdk_util/auto_lock.h
+++ b/native_client_sdk/src/libraries/sdk_util/auto_lock.h
@@ -7,13 +7,22 @@
#define LIBRARIES_SDK_UTIL_AUTO_LOCK_H_
#include <pthread.h>
+#include "sdk_util/macros.h"
+#include "sdk_util/simple_lock.h"
+
+
+// This macro is provided to allow us to quickly instrument locking for
+// debugging perposes.
binji 2013/07/13 00:16:48 sp: purposes
+#define AUTO_LOCK(lock) \
+ AutoLock Lock##__LINE__(lock);
class AutoLock {
public:
- explicit AutoLock(pthread_mutex_t* lock) {
- lock_ = lock;
+ AutoLock(const SimpleLock& lock) {
+ lock_ = lock.mutex();
pthread_mutex_lock(lock_);
}
+
~AutoLock() {
if (lock_) pthread_mutex_unlock(lock_);
}
@@ -25,6 +34,8 @@ class AutoLock {
private:
pthread_mutex_t* lock_;
+
+ DISALLOW_COPY_AND_ASSIGN(AutoLock);
};
#endif // LIBRARIES_SDK_UTIL_AUTO_LOCK_H_

Powered by Google App Engine
This is Rietveld 408576698