On Sat, Mar 06, 2021 at 04:42:35PM -0300, James Almer wrote:
> Signed-off-by: James Almer <jamr...@gmail.com>
> ---
> Changes since v2 is the addition of the buffer_size_t typedef to reduce the
> amount of ifdeffery required to adapt our code.
> 
>  doc/APIchanges              |  4 ++++
>  libavutil/buffer.c          | 14 +++++++-------
>  libavutil/buffer.h          | 32 ++++++++++++++++++++++++++++++++
>  libavutil/buffer_internal.h |  8 ++++----
>  libavutil/internal.h        |  7 +++++++
>  libavutil/version.h         |  5 ++++-
>  6 files changed, 58 insertions(+), 12 deletions(-)

The patchset prior to the bump LGTM
the patchset after the bump (thus enabled size_t) is probably ok but
i did not review this for downstream effects which could introduce bugs
when theres any code that expected the variables to be int

code prior bump tested, code post bump not tested

Thanks

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

The worst form of inequality is to try to make unequal things equal.
-- Aristotle

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to