diff options
| author | Zhao Zhili <zhilizhao@tencent.com> | 2019-12-05 17:02:07 +0800 |
|---|---|---|
| committer | James Almer <jamrial@gmail.com> | 2019-12-05 20:53:52 -0300 |
| commit | 71fd72070c867b4f442e5c71baab9775c51273dc (patch) | |
| tree | 0edb0d05c8a731048b31143928043f0d8d09b478 | |
| parent | 964eb754b456e2bd968b754fd7f3c329264355e4 (diff) | |
| download | ffmpeg-71fd72070c867b4f442e5c71baab9775c51273dc.tar.gz | |
avutil/buffer: use appropriate atomic operations
No functional changes. ref/unref vs add/sub is symmetrical.
Signed-off-by: James Almer <jamrial@gmail.com>
| -rw-r--r-- | libavutil/buffer.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libavutil/buffer.c b/libavutil/buffer.c index 8d1aa5fa84..f0034b026a 100644 --- a/libavutil/buffer.c +++ b/libavutil/buffer.c @@ -116,7 +116,7 @@ static void buffer_replace(AVBufferRef **dst, AVBufferRef **src) } else av_freep(dst); - if (atomic_fetch_add_explicit(&b->refcount, -1, memory_order_acq_rel) == 1) { + if (atomic_fetch_sub_explicit(&b->refcount, 1, memory_order_acq_rel) == 1) { b->free(b->opaque, b->data); av_freep(&b); } @@ -281,7 +281,7 @@ void av_buffer_pool_uninit(AVBufferPool **ppool) pool = *ppool; *ppool = NULL; - if (atomic_fetch_add_explicit(&pool->refcount, -1, memory_order_acq_rel) == 1) + if (atomic_fetch_sub_explicit(&pool->refcount, 1, memory_order_acq_rel) == 1) buffer_pool_free(pool); } @@ -298,7 +298,7 @@ static void pool_release_buffer(void *opaque, uint8_t *data) pool->pool = buf; ff_mutex_unlock(&pool->mutex); - if (atomic_fetch_add_explicit(&pool->refcount, -1, memory_order_acq_rel) == 1) + if (atomic_fetch_sub_explicit(&pool->refcount, 1, memory_order_acq_rel) == 1) buffer_pool_free(pool); } |