[FFmpeg-devel] [PATCH] cmdutils: apply option to codec and format contexts if possible.

[Michael Niedermayer]

On Thu, Oct 18, 2012 at 09:58:39AM +0200, Clément Bœsch wrote:
> On Thu, Oct 18, 2012 at 02:20:58AM +0200, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  cmdutils.c |   19 +++++++++++++------
> >  1 file changed, 13 insertions(+), 6 deletions(-)
> > 
> > diff --git a/cmdutils.c b/cmdutils.c
> > index 0163faf..40b6c31 100644
> > --- a/cmdutils.c
> > +++ b/cmdutils.c
> > @@ -436,6 +436,7 @@ void parse_loglevel(int argc, char **argv, const OptionDef *options)
> >  int opt_default(void *optctx, const char *opt, const char *arg)
> >  {
> >      const AVOption *o;
> > +    int consumed = 0;
> >      char opt_stripped[128];
> >      const char *p;
> >      const AVClass *cc = avcodec_get_class(), *fc = avformat_get_class(), *sc, *swr_class;
> > @@ -447,14 +448,18 @@ int opt_default(void *optctx, const char *opt, const char *arg)
> >      if ((o = av_opt_find(&cc, opt_stripped, NULL, 0,
> >                           AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ)) ||
> >          ((opt[0] == 'v' || opt[0] == 'a' || opt[0] == 's') &&
> > -         (o = av_opt_find(&cc, opt + 1, NULL, 0, AV_OPT_SEARCH_FAKE_OBJ))))
> > +         (o = av_opt_find(&cc, opt + 1, NULL, 0, AV_OPT_SEARCH_FAKE_OBJ)))) {
> >          av_dict_set(&codec_opts, opt, arg, FLAGS);
> > -    else if ((o = av_opt_find(&fc, opt, NULL, 0,
> > -                              AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ)))
> > +        consumed = 1;
> > +    }
> > +    if ((o = av_opt_find(&fc, opt, NULL, 0,
> > +                              AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ))) {
> >          av_dict_set(&format_opts, opt, arg, FLAGS);
> > +        consumed = 1;
> > +    }
> >  #if CONFIG_SWSCALE
> >      sc = sws_get_class();
> > -    if (!o && (o = av_opt_find(&sc, opt, NULL, 0,
> > +    if (!consumed && (o = av_opt_find(&sc, opt, NULL, 0,
> >                           AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ))) {
> >          // XXX we only support sws_flags, not arbitrary sws options
> >          int ret = av_opt_set(sws_opts, opt, arg, 0);
> > @@ -462,21 +467,23 @@ int opt_default(void *optctx, const char *opt, const char *arg)
> >              av_log(NULL, AV_LOG_ERROR, "Error setting option %s.\n", opt);
> >              return ret;
> >          }
> > +        consumed = 1;
> >      }
> >  #endif
> >  #if CONFIG_SWRESAMPLE
> >      swr_class = swr_get_class();
> > -    if (!o && (o = av_opt_find(&swr_class, opt, NULL, 0,
> > +    if (!consumed && (o = av_opt_find(&swr_class, opt, NULL, 0,
> >                                 AV_OPT_SEARCH_CHILDREN | AV_OPT_SEARCH_FAKE_OBJ))) {
> >          int ret = av_opt_set(swr_opts, opt, arg, 0);
> >          if (ret < 0) {
> >              av_log(NULL, AV_LOG_ERROR, "Error setting option %s.\n", opt);
> >              return ret;
> >          }
> > +        consumed = 1;
> >      }
> >  #endif
> >  
> > -    if (o)
> > +    if (consumed)
> >          return 0;
> >      av_log(NULL, AV_LOG_ERROR, "Unrecognized option '%s'\n", opt);
> >      return AVERROR_OPTION_NOT_FOUND;
> 
> Reminds me the timecode thing, which I finally reverted after making in
> distinction between the them; so what is the common
> formats/codecs/whatever options namespace clash this time?

-err_detect

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121018/c558f133/attachment.asc>