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

[Guillaume POIRIER]

Hi,

On 2/17/06, Corey Hickey <bugfood-ml at fatooh.org> wrote:

[..]

> 01-mplayer-dwStart.diff
> * demux_avi sets stream_delay according to dwStart
> * mplayer adjusts audio_delay according to stream_delay
>
> 02-mencoder-dwStart.diff
> * -audio-delay option allows a negative argument (video delay)
> * updated -audio-delay description in man page
> * changed "AUDIO DELAY" in message to "audio delay"
> * added "video delay" message
> * muxer_avi applies stream delay to individual streams as dwStart
>
> 03-ignore-start.diff
> * new option -ignore-start
> * documented -ignore-start in man page
> * mencoder and mplayer respect ignore_start
>
> 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_delays
>
>
> These patches have been sitting around in combined form for a while,
> minus a couple bugfixes, and the only comments are good (thanks to
> Guillaume for testing). The changes are diverse, though, so I'm not
> going to apply them all at once; instead, I will commit each patch, in
> order, no less than 24 hours after the previous one... starting no less
> than 24 hours from now.

If you want, I can test the combined form (since it contains some bug
fixes, it may be interesting to test again), as what really interests
me is the effect of patch 4.

As far as patches 01-03 are concerned, I don't quite know what to look
for, and/or I think that my regular use of MPlayer/MEncoder do not
make me run code paths that your other patches touch.

They are surely worth a try though...

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