[FFmpeg-devel] [PATCH] lavc: remove reset to zero of the AVPicture fields in case of avpicture_alloc() failure

Stefano Sabatini stefasab at gmail.com
Tue Jun 19 23:39:02 CEST 2012


On date Tuesday 2012-06-19 15:03:31 +0200, Michael Niedermayer encoded:
> On Tue, Jun 19, 2012 at 01:07:56PM +0200, Stefano Sabatini wrote:
> > This behavior is not documented, and seems not required in the few places
> > where avpicture_alloc() is used.
> > 
> > Make the behavior of avpicture_alloc() equivalent to that of
> > av_image_alloc(), allowing for the function deprecation and API
> > simplification.
[...]

Forget this patch, it is pointless (I plan to deprecate all the
avpicture_* functions anyway), and *wrong* since av_image_alloc()
doesn't return 0 in case of success.

Dropped.
-- 
FFmpeg = Furious & Faithful Merciful Powerful Erratic Guru


More information about the ffmpeg-devel mailing list