On 10/12/15 22:29, 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. > ---
Sounds good to me. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
