[MPlayer-dev-eng] [PATCH] bframes, dwStart: individual patches

[Guillaume POIRIER]

Hi,

On 2/17/06, Corey Hickey <bugfood-ml at fatooh.org> wrote:
> Corey Hickey wrote:
> > I'm sending this in for review as one big patch because some of the changes
> > I propose don't make much sense out of context of the other changes. Once
> > all the changes look acceptable I intend to propose them as a few smaller,
> > self-contained patches, so please don't tell me to separate them just yet.

[..]

> 04-bframes-delay.diff
> * lavc, xvid, xvid4, and x264 increment encoder_delay on empty frames
> * mencoder adjusts AV_delay according to all encoder_delays
> * lavc, xvid, xvid4, and x264 set decoder_delay appropriately
> * muxer_avi adjusts dwStart according to the decoder_delay

Okay, so the only pending patch is this one. I've had success with the
4 combined since I've been testing them. I've started a new encode
with an NTSC source (I'm on a PAL country) for further testing.

I haven't had a chance to test all the other suggested flavours of
"-delay" options, but when I have more time, I'll do it for sure.

Once again Corey, Kudows for those patches, they are smashing bugs
that have annoyed me for quite a long time, so it"s nice to see them
fixed.

Guillaume
--
Just because code is syntactically "valid" GNU C doesn't mean gcc can
always compile it.
  Steven Bosscher - 2005-01-01
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11203#c14