[MPlayer-dev-eng] [PATCH] Correct rounding for libavcodec/mpeg12.c

[Svante Signell]

On Tue, 2004-02-10 at 19:02, Svante Signell wrote:
> On Tue, 2004-02-10 at 15:25, Michael Niedermayer wrote:
> > Hi
> > 
[...]
> I'll change to 24000/1001 to mencoder when mplayer reports 23.976, and
> similar for 29.97 i.e. 30000/1001=29.97003.
Not even -ofps 24000/1001 works, since the numbers are already quantized
when used in mpegvideo.c. The common denominator to 23976/1000 is 8 i.e.
2997*8/(125*8)=23976/1000, see below:

[mpeg2video @ 0x85790a0]removing common factors from framerate
[mpeg2video @ 0x85790a0]MPEG1/2 doesnt support 2997/125 fps
Could not open codec.

The correct numbers should have been
avctx->frame_rate=24000,avctx->frame_base=1001, not the numbers given
above, 23976 and 1000. Something is fishy here, either in mencoder or
libavcodec...
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng