[FFmpeg-devel] [PATCH] Libavfilter build fix
Vitor Sessak
vitor1001
Fri Feb 29 22:26:18 CET 2008
Hi
V?ctor Paesa wrote:
> Vitor Sessak said:
>> Hi,
>>
>> Mans and vmrsss spotted some bugs in building libavfilter. The patch I
>> attached fix it and also make its build system more in line with the
>> other lav*.
>
>> +find_things2(){
>> + thing=$1
>> + pattern=$2
>> + file=$source_path/$3
>> + sed -n "s/^[^#]*$pattern.*(.*,\\(.*\\),.*).*/\\1_$thing/p" "$file"
>> +}
>
> I am wondering if the filters should follow the suffix tagging style as
> (en)coders/parsers/protocols/(de)muxers, i.e. "movie_vsrc" instead of
> "vsrc_movie" and so on.
Well, now the struct is named avfilter_vsrc_movie. Do you suggest just
movie_vsrc or movie_vsrc_filter? The thing I like in the way it is are
the filenames ("vsrc_movie.c" is tidy when doing ls libavfilter/)...
Irrespective of how we tag it, I don't think it is a good idea to make a
distinction in the configure parameters (having all the
--enable-{video,audio,subtitle,etc}-filter=xxx would be an overkill), so
we'll need a special handling in configure anyway.
-Vitor
More information about the ffmpeg-devel
mailing list