diff options
| author | Timur Pocheptsov <timur.pocheptsov@theqtcompany.com> | 2016-06-29 14:50:34 +0200 |
|---|---|---|
| committer | Timur Pocheptsov <timur.pocheptsov@theqtcompany.com> | 2016-07-19 12:56:12 +0000 |
| commit | 17674ddc637a059c3052953c1ec8a80a7d078293 (patch) | |
| tree | 2a656dafb9dbd4d60c6bc37e6748d3f7e8a0152c /dist | |
| parent | 278fd530f0ad883f88e15d3403f1ab63fbdeb131 (diff) | |
AVFMediaAssetWriter - fix atomics use
1. No need in two different atomics (m_stopped/m_aborted) - the single
one 'm_state' with states (Idle/Active/Aborted) should be enough.
2. QAtomicInt::load/store actually have relaxed memory ordering semantics,
(not like std::atomic with sequential ordering as the default one)
which is not always appropriate - replace with loadAquire/storeRelease instead.
Change-Id: I4ce8c9ca7556de3d2c7e369b8a05276b2870460c
Reviewed-by: Yoann Lopes <yoann.lopes@qt.io>
Diffstat (limited to 'dist')
0 files changed, 0 insertions, 0 deletions