[FFmpeg-devel] [PATCH] Make parse_options take as argument a parse argument function

[Diego Biurrun]

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:

Array

> + * @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.

Diego