[MPlayer-dev-eng] flv A/V sync broken

Michael Niedermayer michaelni at gmx.at
Wed Feb 21 03:50:14 CET 2007


On Tue, Feb 20, 2007 at 07:54:34PM +0200, Uoti Urpala wrote:
> On Tue, 2007-02-20 at 12:07 +0100, Michael Niedermayer wrote:
> > > I checked what was causing it. Seems that demux_lavf sets a fps value
> > > but it doesn't match the timestamps. It sets sh_video->fps to 29.9700298
> > > which makes video.c use a constant 1/fps frame duration, but average
> > > distance between frame timestamps is longer.
> > 
> > ok so what should demux lavf set fps too if not the guessed fps? exact values
> > are not available for several containers
> > and i thought someone said that the fps value doesnt matter ...
> The video.c version has different cases for different demuxers. The
> DEMUXER_TYPE_LAVF code (which differs from others) uses timestamps for
> frame duration only under
> 	if((int)sh_video->fps==1000 || (int)sh_video->fps<=1){

that should be changed, timestamps should always be used, and the corect
ones, dts if mplayer expects dts, pts if it expects pts, feel free to
correct this

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070221/983d5008/attachment.pgp>

More information about the MPlayer-dev-eng mailing list