[FFmpeg-devel] [PATCH 1/4] Remove unnecessary mem.h inclusions

[Anton Khirnov]

Quoting Andreas Rheinhardt (2021-02-04 01:05:05)
> Thanks for the report. I have only checked for whether the relevant
> translation unit uses any of the alloc/free functions (because
> DECLARE_ALIGNED is already provided by mem_internal.h), but I have not
> taken into account stuff that is included by the headers included by
> mem.h. In this case, one needs string.h (which is included in
> libavutil/common.h which in turn is included in avutil.h which is
> included in mem.h). Will look over everything again.
> 
> - Andreas
> 
> PS: It actually seems that the only thing provided by avutil.h that
> mem.h uses is size_t. Do we allow to remove included headers from
> installed headers at major version bumps?

I would say yes, IIRC it has happened before.
Certainly nobody should depend on our headers to provide any random
system headers.

-- 
Anton Khirnov