[FFmpeg-devel] [PATCH] List configurable codecs, (de)muxers, etc.
Panagiotis Issaris
takis.issaris
Fri Jul 27 16:12:46 CEST 2007
Diego Biurrun schreef:
> 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.
>
Applied.
With friendly regards,
Takis
--
vCard: http://issaris.org/pi.vcf
PGP key: http://issaris.org/pi.key
More information about the ffmpeg-devel
mailing list