Re: [FFmpeg-devel] [PATCH] lavc/utils: simplify lockmgr

2017-11-29 Thread Timothy Gu
On Sun, Nov 26, 2017 at 8:44 PM Rostislav Pehlivanov wrote: > @@ -103,8 +104,7 @@ static int default_lockmgr_cb(void **arg, enum > AVLockOp op) > case AV_LOCK_DESTROY: > if (*mutex) > pthread_mutex_destroy(*mutex); > -av_free(*mutex); > -avpriv_atomic_pt

Re: [FFmpeg-devel] [PATCH] lavc/utils: simplify lockmgr

2017-11-27 Thread Michael Niedermayer
On Mon, Nov 27, 2017 at 04:44:19AM +, Rostislav Pehlivanov wrote: > Again, totally unneded use of the atomic function to set/NULL a local > variable. > > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/utils.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) this cras