[MPlayer-cvslog] r25811 - trunk/mencoder.c

[Michael Niedermayer]

On Sun, Jan 20, 2008 at 12:19:42AM +0100, iive wrote:
> Author: iive
> Date: Sun Jan 20 00:19:41 2008
> New Revision: 25811
> 
> Log:
> Extend the precision of rationale conversion so
> it would give proper result for framerate 60000/1001 .
> Otherwise framerate restrained encoders like lavc mpeg2video
> would refuse to encode, even if -ofps 60000/1001 is given.

IIRC this breaks many other framerates (though ive not confirmed it with
the current code)
mencoder uses floats for fps, they just simply dont contain enough
information to reconstruct the original 16/16bit fraction
to fix this fps first must be changed to double

also independant of that float fps is a stupid idea as its just not
accurate enough, converted to fraction or not

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20080120/c1f73b11/attachment.pgp>