[FFmpeg-devel] [PATCH 1/3] avcodec: use av_frame_free() for coded_frame

[Nicolas George]

Le duodi 12 ventôse, an CCXXIII, Clement Boesch a écrit :
> I was confused by the output of FATE, it's not in r210, it's in prores:
>     *avctx->coded_frame           = *pic;

This code is invalid: if a future version of lavu adds fields at the end of
AVFrame, they would not be copied.

(I hoped we could catch all similar cases by adding "struct AVFramePrivate
private;" as the last field, but apparently the C standard does not allow
the last element of a structure to have an incomplete type, even though it
is theoretically possible.)

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150302/84c28cb1/attachment.asc>