Fix unused private field warning of FastPthreadMutex

chenBright · Dec 9, 2024 · 3c3d6cc · 3c3d6cc
1 parent f187d2c
commit 3c3d6cc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/bthread/mutex.cpp b/src/bthread/mutex.cpp
@@ -658,7 +658,7 @@ void submit_contention(const bthread_contention_site_t& csite, int64_t now_ns) {
 }
 
 #if BRPC_DEBUG_LOCK
-#define MUTEX_RESET_OWNER_COMMON(owner)                                                     \
+#define MUTEX_RESET_OWNER_COMMON(owner)                                              \
     ((butil::atomic<bool>*)&(owner).hold)                                            \
         ->store(false, butil::memory_order_relaxed)
 
@@ -678,9 +678,9 @@ void submit_contention(const bthread_contention_site_t& csite, int64_t now_ns) {
                    << std::endl << trace.ToString();                                 \
     }
 #else
-#define MUTEX_RESET_OWNER_COMMON(owner) ((void)0)
-#define PTHREAD_MUTEX_SET_OWNER(owner) ((void)0)
-#define PTHREAD_MUTEX_CHECK_OWNER(owner) ((void)0)
+#define MUTEX_RESET_OWNER_COMMON(owner) ((void)owner)
+#define PTHREAD_MUTEX_SET_OWNER(owner) ((void)owner)
+#define PTHREAD_MUTEX_CHECK_OWNER(owner) ((void)owner)
 #endif // BRPC_DEBUG_LOCK
 
 namespace internal {

diff --git a/src/bthread/mutex.h b/src/bthread/mutex.h
@@ -90,7 +90,7 @@ class FastPthreadMutex {
     // Note: Owner detection of the mutex comes with average execution
     // slowdown of about 50%., so it is only used for debugging and is
     // only available when the macro `BRPC_DEBUG_LOCK' = 1.
-    mutex_owner_t _owner;
+    mutex_owner_t ALLOW_UNUSED _owner;
 };
 #else
 typedef butil::Mutex FastPthreadMutex;