On Wed, Nov 23, 2011 at 08:31:31PM -0800, Ronald S. Bultje wrote: > Hi, > > On Wed, Nov 23, 2011 at 5:01 PM, Janne Grunau <[email protected]> wrote: > > This is required to fulfill avcodec_decode_video2() promise to return > > the number of consumed bytes on success. > > --- > > libavcodec/avcodec.h | 3 ++- > > libavcodec/pthread.c | 5 +++-- > > libavcodec/thread.h | 3 +++ > > 3 files changed, 8 insertions(+), 3 deletions(-) > [..] > > /** > > * Decode the video frame of size avpkt->size from avpkt->data into picture. > > * Some decoders may support multiple frames in a single AVPacket, such > > - * decoders would then just decode the first frame. > > + * decoders would then just decode the first frame. Multiple frames in a > > single > > + * AVPacket are not supported with frame level multithreading. > > That's not true. E.g. concatenated B-frames in AVI-style MPEG-4 is supported.
I'm not sure how those are handled but frame threading can't require the calling code to call decode_video2 again for the second frame. I'll look into it and fix the comment. > I'm not sure if this patch is right yet, what depends on this style of > behaviour? avconv requires this since af8ad892bf846425a86872d01162a59a6a31337c Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
