On Tue, Apr 05, 2011 at 12:55:44PM +0000, Ronald S. Bultje wrote: > 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.
queued Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
