[FFmpeg-devel] [PATCH 1/2] avutil/common: Add FFINCREASE_PTR()

[Nicolas George]

Michael Niedermayer (12021-02-19):
> Suggested-by: Andreas Rheinhardt
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  doc/APIchanges     | 3 +++
>  libavutil/common.h | 2 ++
>  2 files changed, 5 insertions(+)

Is it only for NULL+0 or is it for all NULL+x?

It is ok to hide NULL+0, but NULL+x is a different kind of bug, more
serious, but the macro hides it. Maybe add an av_assert2()?

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210219/f80f2ee5/attachment.sig>