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

[Loren Merritt]

On Mon, 23 Jan 2006, Corey Hickey wrote:

> Corey Hickey wrote:
>> Meanwhile, I'm going to just prepare a patch that makes mencoder's XviD
>> encoding set the encoder delay based on the number of B-frames.
>> Realistically, that should do exactly the same thing. Using a variable
>> set by XviD would just be nicer/cleaner.
>
> Ok, here's the current patch. Unfortunately, it seems for x264 I can't
> access mod->x264->frames.i_delay because x264.h doesn't expose any
> details about the x264_t structure. So, I had to stick with
> mod->param.i_bframe instead.
>
> Does the attached patch look ok? Should I apply it?

I can understand arguments for either bframes or bframes+1, but it is not 
correct to mix them. The decision is a property of AVI, not of the codec. 
H.264 with simple B-frames is exactly the same as MPEG-4 in terms of 
encoding and decoding delay.

And whichever we choose, make sure that MPlayer follows the same rule.

--Loren Merritt