[FFmpeg-soc] [soc]: r316 - in libavfilter: avfilter.c avfilter.h filter_test.c vf_crop.c vo_sdl.c vsrc_dummy.c

[Michael Niedermayer]

Hi

On Thu, Jul 05, 2007 at 05:32:11PM -0400, Bobby Bingham wrote:
> On Thu, 5 Jul 2007 23:16:54 +0200
> Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > Hi
> > 
> > On Thu, Jul 05, 2007 at 10:48:48PM +0200, koorogi wrote:
> > > Author: koorogi
> > > Date: Thu Jul  5 22:48:48 2007
> > > New Revision: 316
> > > 
> > > Log:
> > > Rework filter initialization sequence.  Now supports passing user
> > > parameters, querying supported colorspaces, etc.
> > [...]
> > > @@ -91,6 +92,7 @@ int avfilter_link(AVFilterContext *src, 
> > >                    AVFilterContext *dst, unsigned dstpad)
> > >  {
> > >      AVFilterLink *link;
> > > +    int *fmts[2], i, j;
> > >  
> > 
> > shouldnt this be enum PixelFormat ?
> 
> Maybe.  For the functions where there's a clear analog with audio, I'm
> trying to keep the interface general enough that it could be reused for
> audio filters in case somebody eventually wants to add support for that
> in the libavfilter.  If you prefer that I focus it more on just video
> for the moment, I can certainly change this to enum PixelFormat.

hmm, iam fine with int too


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

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20070705/51c1b9cb/attachment.pgp>