[FFmpeg-devel] [RFC] threading API

[Michael Niedermayer]

On Thu, Oct 08, 2009 at 09:31:02AM +0200, Reimar D?ffinger wrote:
> On Wed, Oct 07, 2009 at 06:13:46PM +0200, Michael Niedermayer wrote:
> > On Tue, Oct 06, 2009 at 04:34:12PM +0200, Reimar D?ffinger wrote:
> > > On Tue, Oct 06, 2009 at 03:23:14PM +0200, Michael Niedermayer wrote:
> > > > maybe we should have 2 arrays, 1 per thread, 1 per task, and they can be NULL
> > > 
> > > Again, that requires an API change. Though on thinking again my approach
> > > isn't really possible without either...
> > 
> > i have no matches in ffmpeg.c for execute() ...
> 
> What does that have to do with it? The documentation for execute says:
>      * The user may replace this with some multithreaded implementation,
>      * the default implementation will execute the parts serially.
>      * @param count the number of things to execute
>      * - encoding: Set by libavcodec, user can override.
>      * - decoding: Set by libavcodec, user can override.
> 
> Apart from the "default implementation will execute the parts serially"
> which is wrong anyway, execute can be set by users, thus changing it is
> an API change.

> Of course it would be possible to just add an execute2 function.

yes


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

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- 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/20091008/4a8fb8d8/attachment.pgp>