[MPlayer-dev-eng] [PATCH] delay-compensated B-frames and dwStart

[Guillaume POIRIER]

Hi,

On 2/13/06, Corey Hickey <bugfood-ml at fatooh.org> wrote:
> Guillaume POIRIER wrote:
> > Corey Hickey wrote:
> >
> >>This patch is the combination of my earlier efforts into (a)fixing video
> >>delay when encoding with B-frames and (b)making mplayer/mencoder
> >>recognize dwStart correctly.
> >
> >
> > [..]
> >
> > All I can say is that if works like a charm on the different files I've
> > encoded with your patch.
>
> Thanks for testing. I've been letting this patch stagnate a while
> because I wanted to get my earlier patch about instant-sync seeking in
> demux_avi committed first -- it makes the operation of the various delay
> options much nicer.
>
> Subject: [PATCH] demux_avi: instant A-V sync correction on seeking
> Date: Thu, 02 Feb 2006 22:17:18 -0800
>
> Since I messed up the commit of that other patch last time I haven't
> threatened to apply the correct one; I was hoping somebody would give it
> their blessing. Since nobody has responded, though, I guess I'll start
> threatening again. If you have time, could you please test it, too?

I can put it on my todo list for sure. However, I must tell you that
no-instant A/V correction on seeking for AVI files have never bothered
me (I wonder if I ever ran into such a file), therefore I'm afraid
I'll just be able to test your patch with the command line you gave in
your mail...

To put it in a nutshell, "delay compensated bframes and dwStart" fixes
a problem that bothered me for a while, whereas your other patch
doesn't fix any problem that I've encountered (I'm not saying that
it's useless, far from it).

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
---
MPlayer's doc isn't up-to-date. Visit my updated mirror here:
http://tuxrip.free.fr//MPlayer-DOCS-HTML/en/
http://tuxrip.free.fr//MPlayer-DOCS-HTML/fr/