On Thu, 10 Dec 2015, Janne Grunau wrote:
Hi,
without comment explaining the reason for the macros.h include in common.h
Janne
--->8---
Include macros.h explicitly in common.h so that external code using
FFALIGN does not break. It was already implicitly included through
version.h. Include macros.h in lls.h and internal.h for FFALIGN.
lls.h was including common.h only for FFALIGN and internal.h was
missing the include for FFALIGN. `make checkheaders` did not catch it
because it's an internal header.
---
libavutil/common.h | 2 +-
libavutil/internal.h | 1 +
libavutil/lls.h | 2 +-
libavutil/macros.h | 2 ++
4 files changed, 5 insertions(+), 2 deletions(-)
Ok with me
// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel