[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