On Sat, 8 Feb 2014, Janne Grunau wrote:
--- libavcodec/arm/dca.h | 1 + libavcodec/x86/dca.h | 1 + 2 files changed, 2 insertions(+)
Perhaps "dca: Make x86/arm dca.h headers self-sufficient" or so?
diff --git a/libavcodec/arm/dca.h b/libavcodec/arm/dca.h index 06e3ea6..580bd75 100644 --- a/libavcodec/arm/dca.h +++ b/libavcodec/arm/dca.h @@ -24,6 +24,7 @@ #include <stdint.h> #include "config.h" +#include "libavcodec/dcadsp.h" #include "libavcodec/mathops.h" #if HAVE_ARMV6_INLINE && AV_GCC_VERSION_AT_LEAST(4,4) diff --git a/libavcodec/x86/dca.h b/libavcodec/x86/dca.h index fbca7ff..ab175b3 100644 --- a/libavcodec/x86/dca.h +++ b/libavcodec/x86/dca.h @@ -23,6 +23,7 @@ #if ARCH_X86_64 && HAVE_SSE2_INLINE # include "libavutil/x86/asm.h" # include "libavutil/mem.h" +#include "libavcodec/dcadsp.h"
Not that we do this in many other places, but you could make the include indentation consistent with the others.
Patch ok with or without these minor nits. // Martin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
