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

[Svante Signell]

On Tue, 2004-02-10 at 20:17, Svante Signell wrote:
> On Tue, 2004-02-10 at 20:07, Svante Signell wrote:
> > On Tue, 2004-02-10 at 19:40, Michael Niedermayer wrote:
> [...]
> > > > Not even -ofps 24000/1001 works, since the numbers are already quantized
> > > cvs -z3 update -d
> > libavcodec or mplayer? I did not find anything immediately in the changed files 
> > libavcodec/adpcm.c,common.c,common.h,g726.c,h263.c,indeo3.c,mjpeg.c,rv10.c
> Sorry, I found it, main/mencoder.c. Thanks.
Unfortunately the latest changes to mplayer-cvs does not work:

 mencoder test1.avi -noskip -ofps 24000/1001 -oac lavc -ovc lavc
-lavcopts
vcodec=mpeg2video:vbitrate=1600:acodec=ac3:abitrate=448:keyint=250:aspect=4/3:vpsize=1000:vrc_maxrate=9352:vrc_buf_size=1853 -of mpeg -o test12.mpg

With cvs version 20040207:
Frame rate already quantized...
[...]
videocodec: libavcodec (640x272 fourcc=3267706d [mpg2])
[mpeg2video @ 0x8575c60]removing common factors from framerate
[mpeg2video @ 0x8575c60]MPEG1/2 doesnt support 2997/125 fps
Could not open codec.
FATAL: Cannot initialize video driver.


With latest cvs version (20040210, i.e. tonight):
Result is a parse error: It seems that the latest changes overwrites
parsing of the next argument.
[...]
No audio encoder (-oac) selected. Select one (see -oac help) or use
-nosound.
Exiting...