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

[Michael Niedermayer]

Hi

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>