[FFmpeg-devel] [PATCH] Make ffplay define in the encoder the?number of threads *before* to open it

Michael Niedermayer michaelni
Mon Dec 28 00:07:57 CET 2009


On Sun, Mar 09, 2008 at 03:54:35PM +0100, Michael Dorr wrote:
> On Sunday 09 March 2008 02:59:30 pm Stefano Sabatini wrote:
> > On date Saturday 2008-03-08 21:03:17 +0100, Michael Niedermayer encoded:
> > > On Sat, Mar 08, 2008 at 06:33:27PM +0100, Stefano Sabatini wrote:
> > > [...
> > >
> > > > Index: ffplay.c
> > > > -    enc->thread_count= thread_count;
> 
> [...]
> 
> > Without threads enabled enc->thread_count isn't set, so it defaults to the
> > sane value of 1.
> 
> I hope this doesn't qualify as thread hijacking, but the same patch (remove 
> setting enc->thread_count) should also be applied to ffmpeg.c (attached).
> 
> Regards
> 
> -- 
> Michael

>  ffmpeg.c |    3 ---
>  1 file changed, 3 deletions(-)
> e0ef035296de4281042fdb7d73f17b33267aa132  donotset_threadcount_ffmpeg.diff

ok if tested and multitheraded decoding AND encoding still works

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091228/c17663bd/attachment.pgp>



More information about the ffmpeg-devel mailing list