[FFmpeg-devel] [RFC][PATCH 1/4] avutil/frame: add an internal field to store the size of AVFrame

Michael Niedermayer michael at niedermayer.cc
Sat Feb 12 14:08:02 EET 2022


On Fri, Feb 11, 2022 at 09:12:58PM -0300, James Almer wrote:
> This is unfortunately needed to remove (or reduce the awfulness) of certain
> modules violating the AVFrame API and using sizeof(AVFrame).
> With this, the sizeof(AVFrame) value of the libavutil loaded at runtime can be
> used instead of the compile time value of whatever library included frame.h
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> This is sucks, but at least less so than the current situation.
> 
> I don't see wrapped_avframe going away anytime soon, so something must be done,
> and last time i tried to change how the packets are generated my approach was
> shut down, so here's another attempt.

iam probably missing something but if the goal is to wrap AVFrame in some
other structure as a array or buffer
without the sizeof(AVFrame) cant the wraping/unwraping code be put in
libavutil ?

thx

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

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20220212/56751b2d/attachment.sig>


More information about the ffmpeg-devel mailing list