[FFmpeg-devel] [PATCH] Remove some dependancies of libavfilter on libavcodec

Stefano Sabatini stefano.sabatini-lala
Sun Jan 31 17:23:54 CET 2010


On date Sunday 2010-01-31 16:54:43 +0100, Michael Niedermayer encoded:
> On Sun, Jan 31, 2010 at 03:19:13PM +0100, Stefano Sabatini wrote:
> > Hi,
> > avoid the use of avcodec_get_pix_fmt_name() and
> > avcodec_get_chroma_sub_sample(), which are not necessary anymore since
> > the pixdesc.h API addition.
> > 
> > Alternatively we could have the functions:
> > av_get_pix_fmt_chroma_shift()
> > 
> > and maybe a:
> > av_get_pix_fmt_name()
> > 
> > which may be used by other librearies as well (lsws).
> > 
> 
> > Apart from these, lavfi uses from lavc still the enum:
> > enum CodecType (avcodec.h)
> 
> and where is the problem? thats a compile time dependancy on a header
> not a runtime dependancy on a lib

An user may want to install lavfi and not lavc, that is currently
possible with the new --disable-avcodec switch.

Then she tryies to compile a program which requires lavfi, it will
fail as the lavc headers required by lavfi have not been installed. 

> > ff_fill_linesize() (imgconvert.h)
> > ff_fill_pointer() (imgconvert.h)
> > 
> > would be OK to move these to lavu as well?
> 
> uhm, iam definitly not happy about this, no

What do you suggest to do instead?

[...]

Regards.
-- 
FFmpeg = Fast & Fast Mega Power Elfic Gadget



More information about the ffmpeg-devel mailing list