[SCM] libav/experimental: pthread_frame: flush all threads on flush, not just the first one

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Aug 10 16:01:46 UTC 2014


The following commit has been merged in the experimental branch:
commit d1f9563d502037239185c11578cc614bdf0c5870
Author: Anton Khirnov <anton at khirnov.net>
Date:   Thu Feb 27 21:36:33 2014 +0100

    pthread_frame: flush all threads on flush, not just the first one
    
    avcodec_flush_buffers() must release all internally held references
    according to its documentation, for which all the threads need to be
    flushed.
    
    CC:libav-stable at libav.org
    Bug-Id: vlc/9665

diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c
index 36b4fff..1af8ff5 100644
--- a/libavcodec/pthread_frame.c
+++ b/libavcodec/pthread_frame.c
@@ -672,8 +672,6 @@ void ff_thread_flush(AVCodecContext *avctx)
     if (fctx->prev_thread) {
         if (fctx->prev_thread != &fctx->threads[0])
             update_context_from_thread(fctx->threads[0].avctx, fctx->prev_thread->avctx, 0);
-        if (avctx->codec->flush)
-            avctx->codec->flush(fctx->threads[0].avctx);
     }
 
     fctx->next_decoding = fctx->next_finished = 0;
@@ -686,6 +684,9 @@ void ff_thread_flush(AVCodecContext *avctx)
         av_frame_unref(p->frame);
 
         release_delayed_buffers(p);
+
+        if (avctx->codec->flush)
+            avctx->codec->flush(p->avctx);
     }
 }
 

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list