On Sat, Jan 28, 2012 at 07:03:54PM +0100, Kostya Shishkov wrote:
> On Sat, Jan 21, 2012 at 09:49:23PM +0000, Alexandra Khirnova wrote:
> > --- a/cmdutils.c
> > +++ b/cmdutils.c
> > @@ -288,7 +288,7 @@ void parse_options(void *optctx, int argc, char **argv, 
> > const OptionDef *options
> >                     void (* parse_arg_function)(void *, const char*))
> >  {
> >      const char *opt;
> > -    int optindex, handleoptions = 1, ret;
> > +    int optindex, handleoptions = 1, ret, last_parse_arg_index = 0;
> >  
> > @@ -311,8 +311,16 @@ void parse_options(void *optctx, int argc, char 
> > **argv, const OptionDef *options
> >          } else {
> >              if (parse_arg_function)
> >                  parse_arg_function(optctx, opt);
> > +            last_parse_arg_index = optindex;
> >          }
> >      }
> > +    if (optindex - last_parse_arg_index) {
> > +        int i;
> > +        av_log(NULL, AV_LOG_WARNING, "Trailing options found: ");
> > +        for (i = optindex - last_parse_arg_index; i > 0; i--)
> > +            av_log(NULL, AV_LOG_WARNING, "%s ", argv[argc - i]);
> > +        av_log(NULL, AV_LOG_WARNING, "they will be ignored.\n");
> > +    }
> 
> probably OK

This was never applied and does not apply cleanly anymore.

Alexandra, could you rebase it?

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to