[FFmpeg-devel] [PATCH] Define AVMediaType in libavutil

[Stefano Sabatini]

On date Saturday 2010-03-13 19:13:44 +0100, Michael Niedermayer encoded:
> On Sat, Mar 13, 2010 at 04:10:04PM +0100, Stefano Sabatini wrote:
> > Hi, see patch description.
> > 
> > This is meant mainly to remove one of the *compilation-time*
> > dependancy of libavfilter on libavcodec.
> > 
> > I hope it is clear why this is desirable (note: currently we can
> > already have the libavutil+libavfilter headers installed and not the
> > libavcodec ones).
> > 
> > If this is OK I can replace all occurrences of CodecType with
> > corresponding AVMediaType symbols and drop the CodecType definition,
> > 
> > *or*
> > 
> > keep an independent definition of CodecType.
> > 
> > Regards.
> > -- 
> > FFmpeg = Faithful & Fundamental Mortal Portentous Exciting Geek
> 
> >  libavcodec/avcodec.h |   18 +++++++++---------
> >  libavutil/avutil.h   |   12 +++++++++++-
> >  2 files changed, 20 insertions(+), 10 deletions(-)
> > 23c583d2485a356073472b3805cec69d9b60bef2  0001-Define-enum-AVMediaType-and-make-enum-CodecType-depe.patch
> > >From b007b145584c1e86eb3c10cc9b49f00cb12f18b5 Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > Date: Sat, 13 Mar 2010 16:01:06 +0100
> > Subject: [PATCH] Define enum AVMediaType and make enum CodecType depends on that.
> 
> iam mildly against this patch

Please elaborate.
-- 
FFmpeg = Fundamentalist and Foolish Minimal Peaceful Enlightened Geisha