diff options
| author | Matt Peters <[email protected]> | 2021-12-03 08:22:13 -0700 |
|---|---|---|
| committer | Matt Peters <[email protected]> | 2021-12-03 08:22:13 -0700 |
| commit | 0d8d8b09f925bd8bd0183c647e21e209ed71284b (patch) | |
| tree | 23dac3bb52fdd3311f61370702c6e0dfe6527a48 /zencore/include | |
| parent | Make IoBufferCore::Materialize threadsafe (diff) | |
| download | zen-0d8d8b09f925bd8bd0183c647e21e209ed71284b.tar.xz zen-0d8d8b09f925bd8bd0183c647e21e209ed71284b.zip | |
Fixes from code review
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 |