[FFmpeg-devel] [PATCH] Implement avfilter_add_colorspaces()

[Michael Niedermayer]

On Thu, Dec 31, 2009 at 01:48:29AM +0100, Stefano Sabatini wrote:
> On date Thursday 2009-12-31 01:18:27 +0100, Michael Niedermayer encoded:
> > On Wed, Dec 30, 2009 at 01:32:36AM +0100, Stefano Sabatini wrote:
> [...]
> > > Index: libavfilter-soc/ffmpeg/libavfilter/avfilter.h
> > > ===================================================================
> > > --- libavfilter-soc.orig/ffmpeg/libavfilter/avfilter.h	2009-12-30 01:24:53.000000000 +0100
> > > +++ libavfilter-soc/ffmpeg/libavfilter/avfilter.h	2009-12-30 01:25:01.000000000 +0100
> > > @@ -192,6 +192,12 @@
> > >  AVFilterFormats *avfilter_make_format_list(const enum PixelFormat *pix_fmts);
> > >  
> > >  /**
> > > + * Adds pix_fmt to the list of formats pointed by formats, and updates
> > > + * formats to point to the new updated list.
> > > + */
> > > +void avfilter_add_colorspace(AVFilterFormats **formats, enum PixelFormat pix_fmt);
> > 
> > why **formats and not *formats ?
> 
> The list of formats is re-allocated, so its address may
> change. Alternatively we could have:

formats->formats is reallocated, i dont see formats being

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

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- 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/20091231/19cfee2d/attachment.pgp>