[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