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

Panagiotis Issaris takis.issaris
Fri Jul 27 15:49:22 CEST 2007


Hi Diego,

Diego Biurrun schreef:
> 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.  

Fixed.

> Maybe you could use "part" or
> something better instead of "i" as variable name, but I don't much care.

Fixed.


> Patch OK with me otherwise.


configure |   19 +++++++++++++++++++
 1 files changed, 19 insertions(+)


With friendly regards,
Takis
--
vCard: http://issaris.org/pi.vcf
PGP key: http://issaris.org/pi.key
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: pi-20070727T154648-ffmpeg-configure_lists3.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070727/0717d070/attachment.txt>



More information about the ffmpeg-devel mailing list