[MPlayer-dev-eng] [PATCH] mencoder, B-frames, and video/audio delay

[Loren Merritt]

On Sat, 21 Jan 2006, Corey Hickey wrote:
> Rich Felker wrote:
>
>>> --- libmpcodecs/ve_lavc.c	10 Jan 2006 11:09:02 -0000	1.125
>>> +++ libmpcodecs/ve_lavc.c	21 Jan 2006 05:44:32 -0000
>>> @@ -750,6 +750,9 @@
>>>         mux_v->bih->biSize= sizeof(BITMAPINFOHEADER) + lavc_venc_context->extradata_size;
>>>     }
>>>
>>> +    if (lavc_venc_context->max_b_frames)
>>> +        mux_v->encoder_delay = lavc_venc_context->max_b_frames + 1;
>>> +
>>
>> Why the +1? This seems incorrect,
>
> I don't know. That's what Michael suggested and I've been assuming it to
> be correct. I know I can't really notice 1/24 second sync difference.
>
> http://www1.mplayerhq.hu/pipermail/mplayer-dev-eng/2005-December/039425.html

I think the +1 is for decoding delay. So it is true that total delay = 
bframes+1, but it is incorrect to put the +1 in the muxer.

--Loren Merritt