[FFmpeg-devel] [PATCH] Make cmdutils.c:parse_number_or_die() use av_strtod()

Stefano Sabatini stefano.sabatini-lala
Sun Jun 21 12:40:18 CEST 2009


Hi, as in subject.

This would allow for example to use -ar 44K or -fs 10Mi.

I'm aware that in theory av_strtod() is not part of the public API,
anyway non-public functions are already used in the ff* tools
(e.g. ff_random_get_seed() in ffserver.c), also I'm still of the
opinion that eval should be moved to libavutil and exported.

BTW Michael proposed to use here full expression parsing here, check
for example check this:
http://thread.gmane.org/20080820093923.GE31819 at geppetto

and the second attached patch.

Would be that preferable?

Regards.
-- 
FFmpeg = Frightening & Faboulous Mortal Perennial Enhancing Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cmdutils-use-avstrtod.patch
Type: text/x-diff
Size: 691 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090621/6d397388/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: use-ff-eval-in-cmdutils.patch
Type: text/x-diff
Size: 1195 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090621/6d397388/attachment-0001.patch>



More information about the ffmpeg-devel mailing list