diff options
| author | Chris Robinson <chris.kcat@gmail.com> | 2018-12-30 21:58:14 -0800 |
|---|---|---|
| committer | Chris Robinson <chris.kcat@gmail.com> | 2018-12-30 21:58:14 -0800 |
| commit | 5e03941701c112083b5dce14257fd8c51262f04f (patch) | |
| tree | e60fe1ba85f1d42e0f2b574360713842769df135 /Alc/backends/opensl.cpp | |
| parent | 9f5c9a2260849240bd680b8fdf533acdef7f9de1 (diff) | |
Use an atomic bool on things that only take true or false
Diffstat (limited to 'Alc/backends/opensl.cpp')
| -rw-r--r-- | Alc/backends/opensl.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Alc/backends/opensl.cpp b/Alc/backends/opensl.cpp index 766b7675..69f9cda4 100644 --- a/Alc/backends/opensl.cpp +++ b/Alc/backends/opensl.cpp @@ -169,7 +169,7 @@ struct OpenSLPlayback final : public BackendBase { ALsizei mFrameSize{0}; - std::atomic<ALenum> mKillNow{AL_TRUE}; + std::atomic<bool> mKillNow{true}; std::thread mThread; static constexpr inline const char *CurrentPrefix() noexcept { return "OpenSLPlayback::"; } @@ -550,7 +550,7 @@ ALCboolean OpenSLPlayback::start() if(SL_RESULT_SUCCESS != result) return ALC_FALSE; try { - mKillNow.store(AL_FALSE); + mKillNow.store(false, std::memory_order_release); mThread = std::thread(std::mem_fn(&OpenSLPlayback::mixerProc), this); return ALC_TRUE; } @@ -564,7 +564,7 @@ ALCboolean OpenSLPlayback::start() void OpenSLPlayback::stop() { - if(mKillNow.exchange(AL_TRUE) || !mThread.joinable()) + if(mKillNow.exchange(true, std::memory_order_acq_rel) || !mThread.joinable()) return; mSem.post(); |