[FFmpeg-devel] [PATCH] cmdutils: make parse_options() accept a parse_arg_function with generic signature

Stefano Sabatini stefasab at gmail.com
Sun Jan 15 11:37:53 CET 2012


On date Thursday 2012-01-12 18:25:27 +0100, Stefano Sabatini encoded:
> Make parse_options() accept a parse_arg_function with signature:
> int (* parse_arg_function)(void *optctx, const char *, const char*)
> rather than:
> int (* parse_arg_function)(void *optctx, const char*)
> 
> This is required as this function may be used for other options as well,
> as in the case of the ffplay and ffprobe -i options.
> 
> In particular, fix ffprobe -i FILE behavior.
> ---
>  avconv.c   |    3 ++-
>  cmdutils.c |    4 ++--
>  cmdutils.h |    2 +-
>  ffmpeg.c   |    3 ++-
>  ffplay.c   |    9 ++++-----
>  ffprobe.c  |    7 ++++---
>  6 files changed, 15 insertions(+), 13 deletions(-)

Updated, with bonus patch.
-- 
FFmpeg = Furious and Fostering Mega Purposeless Eccentric Governor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-cmdutils-slightly-simplify-branching-logic-in-parse_.patch
Type: text/x-diff
Size: 890 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120115/775210b9/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-cmdutils-make-parse_options-accept-a-parse_arg_funct.patch
Type: text/x-diff
Size: 6501 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120115/775210b9/attachment-0001.bin>


More information about the ffmpeg-devel mailing list