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

[Michael Niedermayer]

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



> 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


> 
> Regards.
> -- 
> FFmpeg = Frenzy and Fundamental Magical Programmable Exploitable Gadget

>  avfilter.c   |    6 ++++--
>  vf_scale.c   |    6 ++++--
>  vf_slicify.c |    4 ++--
>  vf_vflip.c   |    4 ++--
>  4 files changed, 12 insertions(+), 8 deletions(-)
> 2f6943ef4687bb4b7ef34aa0c6ca5c210a1a05b9  rm-some-lavc-deps-from-lavfi.patch

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100131/ac2c85e9/attachment.pgp>