[MPlayer-dev-eng] [PATCH] twopass xvid encoding of indata with multiple headers

[Aurelien Jacobs]

On Sun, 5 Dec 2004 15:40:47 -0500
D Richard Felker III <dalias at aerifal.cx> wrote:

> On Sun, Dec 05, 2004 at 07:34:43PM +0100, Reimar Döffinger wrote:
> > Hi,
> > > >When I was trying to encode my "Finding Nemo" dvd with mencoder
> > > >and two pass xvid encoding the the actual bitrate of the
> > > >resulting file was way off.
> > > >
> > > >I tracked it down to the fact that when decoding the mpeg2 stream
> > > >from the dvd it encounters multiple "header" segments in the
> > > >datastream (I'm unfamiliar with the correct terminology but I
> > > >hope you get the idea). When mpeg2_parse() in
> > > >libmpcodecs/vd_libmpeg2.c:decode() returns STATE_SEQUENCE it
> > > >triggers a call to xvid_encore() with second argument
> > > 
> > > is it really necessary that vd_libmpeg2.c re-init libvo for every 
> > > STATE_SEQUENCE?
> > 
> > >From what I can tell it actually only makes sense when some of the
> > parameters (width, height, fps) have changed...
> 
> agree. actually only width, height. fps change is irrelevant to
> mplayer.

agree. But maybe we could check if width and height have changed and
only in that case reinit vo ?

Aurel