[FFmpeg-devel] [PATCH] Make parse_options take as argument a parse argument function
Mon Aug 13 20:10:38 CEST 2007
On Mon, Aug 13, 2007 at 11:06:01AM +0200, Stefano Sabatini wrote:
> Sorry for insisting, can someone apply this (I have other patches
> waiting for it)?
I will in a moment, but please clarify the log message:
make cmdutils.c:parse_options take as argument a parse bare argument function.
What exactly do you want that function to take as argument? Is the
following still an accurate description of your change?
Make cmdutils.c:parse_options accept a function to parse bare
parameters as argument.
> --- cmdutils.h (revision 10102)
> +++ cmdutils.h (working copy)
> @@ -50,8 +50,16 @@
> + * Parses the command line arguments.
> + * @param options array with the definitions required to interpret every option of the form:
> + * @param parse_arg_function name of the function to be called to process an argument without
Name of the function called to process an argument without
> + * a leading option name flag, NULL if such argument don't have to be processed.
flag. NULL if such an argument does not have to be processed.
> + */
> +void parse_options(int argc, char **argv, const OptionDef *options, void (* parse_arg_function)(const char*));
And please try to keep lines below 80 characters.
More information about the ffmpeg-devel