[FFmpeg-devel] [PATCH] Add a dummy option -i to ffplay so that it is easy to switch between ffmpeg -i "file" and ffplay -i "file".

Stefano Sabatini stefano.sabatini-lala
Tue Mar 8 15:49:52 CET 2011


On date Tuesday 2011-03-08 15:29:46 +0100, banan at ludd.ltu.se encoded:
> From: Benjamin Larsson <benjamin at southpole.se>
> 
> ---
>  cmdutils.c |    2 ++
>  cmdutils.h |    1 +
>  ffplay.c   |    1 +
>  3 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/cmdutils.c b/cmdutils.c
> index 514ebad..e4ae93c 100644
> --- a/cmdutils.c
> +++ b/cmdutils.c
> @@ -214,6 +214,8 @@ unknown_opt:
>                      fprintf(stderr, "%s: failed to set value '%s' for option '%s'\n", argv[0], arg, opt);
>                      exit(1);
>                  }
> +            } else if (po->flags & OPT_DUMMY) {
> +                /* Do nothing for this option */
>              } else {
>                  po->u.func_arg(arg);
>              }
> diff --git a/cmdutils.h b/cmdutils.h
> index c3d8a42..01f7437 100644
> --- a/cmdutils.h
> +++ b/cmdutils.h
> @@ -122,6 +122,7 @@ typedef struct {
>  #define OPT_FUNC2  0x0400
>  #define OPT_INT64  0x0800
>  #define OPT_EXIT   0x1000
> +#define OPT_DUMMY  0x2000
>       union {
>          void (*func_arg)(const char *); //FIXME passing error code as int return would be nicer then exit() in the func
>          int *int_arg;
> diff --git a/ffplay.c b/ffplay.c
> index e71c9c9..9d66572 100644
> --- a/ffplay.c
> +++ b/ffplay.c
> @@ -3019,6 +3019,7 @@ static const OptionDef options[] = {
>  #endif
>      { "rdftspeed", OPT_INT | HAS_ARG| OPT_AUDIO | OPT_EXPERT, {(void*)&rdftspeed}, "rdft speed", "msecs" },
>      { "default", OPT_FUNC2 | HAS_ARG | OPT_AUDIO | OPT_VIDEO | OPT_EXPERT, {(void*)opt_default}, "generic catch all option", "" },
> +    { "i", OPT_DUMMY, NULL, "ffmpeg compatibility dummy option", ""},
>      { NULL, },
>  };

Missing doc/ffplay.texi changes.
-- 
FFmpeg = Forgiving & Faithless Maxi Plastic Evil Gnome



More information about the ffmpeg-devel mailing list