Hi,

On Thu, Nov 10, 2011 at 1:02 PM, Janne Grunau <[email protected]> wrote:
> this fixes fate-iv8-demux with two threads
> ---8<---
> Adding the thread count in frame level multithreading to has_b_frames
> as an additional delay causes more problems than it solves.
> For example inconsistent behaviour during timestamp calculation in
> libavformat.
> Thread count and frame level multithreading are both set by the user.
> If the additional delay caused by frame level multithreading needs
> to be considered in the calling code it has all information to take
> it into account.
> Should it become necessary to calculate a maximum delay inside
> libavcodec it should be exported as its own field and not reusing
> an existing field.
>
> Based on a patch by Michael Niedermayer.
> ---
>  libavcodec/pthread.c |    4 +---
>  libavcodec/version.h |    2 +-
>  libavformat/utils.c  |    5 -----
>  3 files changed, 2 insertions(+), 9 deletions(-)

I have no objection to this.

Alexander, any comments?

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to