Hi, On Sat, Apr 2, 2011 at 12:17 AM, Alexander Strange <[email protected]> wrote: > On Tue, Mar 29, 2011 at 9:43 PM, Ronald S. Bultje <[email protected]> wrote: >> On Tue, Mar 29, 2011 at 2:23 PM, Alexander Strange >> <[email protected]> wrote: >>> thread_count passed to ff_thread_init() is only used to set AVCodecContext. >>> thread_count, and can be removed. Instead move it to the legacy >>> implementation >>> of avcodec_thread_init(). >>> >>> This also fixes the problem that calling avcodec_thread_init() with pthreads >>> enabled did not set it since ff1efc524cb3c60f2f746e3b4550bb1a86c65316. >>> --- >>> libavcodec/pthread.c | 2 +- >>> libavcodec/thread.h | 2 +- >>> libavcodec/utils.c | 8 ++++---- >>> libavcodec/w32thread.c | 3 +-- >>> 4 files changed, 7 insertions(+), 8 deletions(-) >> >> Looks OK to me, but I can't test on win32 for you... > > I saw an equivalent patch on ffmpeg, with an equivalent change to > w32thread, which was wrong, and a followup fix. Rebased the fix onto > this one. > > I think it's correct now.
Patch is OK, can someone apply? Otherwise I'll apply tonight. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
