diff options
Diffstat (limited to 'zencore/include')
| -rw-r--r-- | zencore/include/zencore/iobuffer.h | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/zencore/include/zencore/iobuffer.h b/zencore/include/zencore/iobuffer.h index a39f442e3..9d18a55e9 100644 --- a/zencore/include/zencore/iobuffer.h +++ b/zencore/include/zencore/iobuffer.h @@ -112,7 +112,7 @@ public: inline void EnsureDataValid() const { - uint32_t LocalFlags = m_Flags.load(std::memory_order_acquire); + const uint32_t LocalFlags = m_Flags.load(std::memory_order_acquire); if ((LocalFlags & kIsExtended) && !(LocalFlags & kIsMaterialized)) { Materialize(); @@ -193,12 +193,11 @@ public: { ZEN_ASSERT_SLOW((uint32_t(ContentType) & kContentTypeMask) == uint32_t(ContentType)); uint32_t OldValue = m_Flags.load(std::memory_order_relaxed); - bool bSucceeded; + uint32_t NewValue; do { - uint32_t NewValue = (OldValue & ~(kContentTypeMask << kContentTypeShift)) | (uint32_t(ContentType) << kContentTypeShift); - bSucceeded = m_Flags.compare_exchange_weak(OldValue, NewValue, std::memory_order_relaxed, std::memory_order_relaxed); - } while (!bSucceeded); + NewValue = (OldValue & ~(kContentTypeMask << kContentTypeShift)) | (uint32_t(ContentType) << kContentTypeShift); + } while (!m_Flags.compare_exchange_weak(OldValue, NewValue, std::memory_order_relaxed, std::memory_order_relaxed)); } inline ZenContentType GetContentType() const |