[FFmpeg-devel] [PATCH] List configurable codecs, (de)muxers, etc.
Diego Biurrun
diego
Fri Jul 27 14:47:10 CEST 2007
On Wed, May 23, 2007 at 01:45:23PM +0200, Panagiotis Issaris wrote:
>
> 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?
Just --list-encoders.
> --- configure (revision 9104)
> +++ configure (working copy)
> @@ -922,6 +929,13 @@
>
> +show_list() {
> + for i in $*; do
> + echo $i|sed 's/_[^_]*$//'
Please leave spaces around the pipe. Maybe you could use "part" or
something better instead of "i" as variable name, but I don't much care.
Patch OK with me otherwise.
Diego
More information about the ffmpeg-devel
mailing list