[FFmpeg-devel] [PATCH] List configurable codecs, (de)muxers, etc.

[Diego Biurrun]

On Fri, Jul 27, 2007 at 04:07:54PM +0200, Panagiotis Issaris wrote:
> 
> M?ns Rullg?rd schreef:
> > Panagiotis Issaris <takis.issaris at uhasselt.be> writes:
> > 
> >> M?ns Rullg?rd wrote:
> >>> Aurelien Jacobs <aurel at gnuage.org> writes:
> >>>
> >>>> Might be simplified using something like:
> >>>>   --list-*)
> >>>>   NAME=`echo "$opt" | sed......
> >>>>   NAME=`toupper $NAME`
> >>>>   show_list `eval \$${NAME}_LIST`
> >>> My thoughts as well.
> >> The attached patch takes Aurelien's suggestion into account.
> >>
> >> Currently it allows both:
> >> --list-encoder
> >> and
> >> --list-encoders
> >>
> >> While the --help info shows:
> >> --list-encoders
> >>
> >> As I found that nicer, although possibly a bit less consistent. What's
> >> your preference?
> >>
> >> --- configure	(revision 9104)
> >> +++ configure	(working copy)
> >> @@ -997,6 +1011,11 @@
> >>    echo "$CMDLINE_SELECT" | grep -q "^ *$option\$" || die_unknown $opt
> >>    $action $option
> >>    ;;
> >> +  --list-*)
> >> +    NAME=`echo "$opt" | sed 's/--list-//;s/s$//;s/$/_LIST/'`
> >> +    NAME=`toupper $NAME`
> >> +    show_list `eval echo \\$${NAME}`
> > 
> > This (untested) looks nicer to me:
> > 
> > NAME="${opt#--list-}"
> > NAME=$(toupper ${NAME%s})
> > eval show_list \$${NAME}_LIST
> 
> I have no preference in this, so here's modified patch.

Commit it.

Diego