[libav-devel] [PATCH v2 1/1] libavutil: move FFALIGN macro from common.h to macros.h

Martin Storsjö martin at martin.st
Fri Dec 11 09:00:58 CET 2015


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


More information about the libav-devel mailing list