diff options
author | Chris Kennelly <ckennelly@google.com> | 2024-04-25 08:57:21 -0700 |
---|---|---|
committer | Copybara-Service <copybara-worker@google.com> | 2024-04-25 08:58:25 -0700 |
commit | 8a3ae1b6f5fb8a034408cc665f4ce1bf7a5e190d (patch) | |
tree | b69665401954e02da5a45ef2b356a5b997a94563 /absl/base | |
parent | 08b21bd037990c18d44fda1691211e73835bf214 (diff) |
Apply ABSL_MUST_USE_RESULT to try lock functions.
PiperOrigin-RevId: 628091370
Change-Id: I2dd20b7f33ab99e78d63688832ab475a513aa3fd
Diffstat (limited to 'absl/base')
-rw-r--r-- | absl/base/internal/spinlock.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/absl/base/internal/spinlock.h b/absl/base/internal/spinlock.h index 301c3e3b..1bb260f4 100644 --- a/absl/base/internal/spinlock.h +++ b/absl/base/internal/spinlock.h @@ -89,7 +89,8 @@ class ABSL_LOCKABLE ABSL_ATTRIBUTE_WARN_UNUSED SpinLock { // acquisition was successful. If the lock was not acquired, false is // returned. If this SpinLock is free at the time of the call, TryLock // will return true with high probability. - inline bool TryLock() ABSL_EXCLUSIVE_TRYLOCK_FUNCTION(true) { + ABSL_MUST_USE_RESULT inline bool TryLock() + ABSL_EXCLUSIVE_TRYLOCK_FUNCTION(true) { ABSL_TSAN_MUTEX_PRE_LOCK(this, __tsan_mutex_try_lock); bool res = TryLockImpl(); ABSL_TSAN_MUTEX_POST_LOCK( @@ -120,7 +121,7 @@ class ABSL_LOCKABLE ABSL_ATTRIBUTE_WARN_UNUSED SpinLock { // Determine if the lock is held. When the lock is held by the invoking // thread, true will always be returned. Intended to be used as // CHECK(lock.IsHeld()). - inline bool IsHeld() const { + ABSL_MUST_USE_RESULT inline bool IsHeld() const { return (lockword_.load(std::memory_order_relaxed) & kSpinLockHeld) != 0; } |