[FFmpeg-cvslog] r23009 - trunk/libavcodec/avcodec.h

[Reimar Döffinger]

On Mon, May 03, 2010 at 01:27:56PM +0200, Michael Niedermayer wrote:
> On Sun, May 02, 2010 at 06:07:30PM +0200, reimar wrote:
> > Author: reimar
> > Date: Sun May  2 18:07:29 2010
> > New Revision: 23009
> > 
> > Log:
> > Clarify how allocation works for the picture argument for
> > avcodec_decode_video3.
> > 
> > Modified:
> >    trunk/libavcodec/avcodec.h
> > 
> > Modified: trunk/libavcodec/avcodec.h
> > ==============================================================================
> > --- trunk/libavcodec/avcodec.h	Sat May  1 23:05:28 2010	(r23008)
> > +++ trunk/libavcodec/avcodec.h	Sun May  2 18:07:29 2010	(r23009)
> > @@ -3464,6 +3464,8 @@ attribute_deprecated int avcodec_decode_
> >   *
> >   * @param avctx the codec context
> >   * @param[out] picture The AVFrame in which the decoded video frame will be stored.
> > + *             Use avcodec_alloc_frame to get an AVFrame, the codec will call
> 
> > + *             get_buffer to get memory for the actual image data.
> 
> not all codecs will

Well, I think there are only 2 exception, schroedinger and qtrleenc.
And I thought the idea was that they should use get_buffer as well
(though only work with the default ones that do whatever special tricks
necessary).
What would you prefer?
", the codec will itself allocate the memory for the actual image data."
Maybe add a "(e.g. via get_buffer)"?