[FFmpeg-devel] [PATCH] Null lavfi source

Stefano Sabatini stefano.sabatini-lala
Thu Jan 7 00:23:08 CET 2010


On date Wednesday 2010-01-06 23:50:37 +0100, Michael Niedermayer encoded:
> On Wed, Jan 06, 2010 at 08:56:13PM +0100, Stefano Sabatini wrote:
> > Hi, this is a pre-requisite for the graph2dot tool, so it may be
> > useful for other similar debug/analysis tools.
> > 
> > Note that avcodec.h is included just because
> > av_parse_frame_{rate,size} is used (so I suggest to move them to lavu
> > at the next major bump).
> 
> i think we do not need that, this is not a filter likely used by users
> directly thus its all right if it supports just numbers and not
> adds a dependancy between libs or crap to lavu. Not to mention other
> filters also work with just numbers and no scale=cif

I don't know, we could avoid for sure av_parse_frame_{size,rate} in
this filter but those functions are potentially useful even outside
lavc, any source filter is a good candidate to use it.

Regards.
-- 
FFmpeg = Faboulous & Faboulous Most Problematic Elected Gargoyle



More information about the ffmpeg-devel mailing list