From 4b357bd15ba6a45687a4b01b14bca46e6cbe6cd2 Mon Sep 17 00:00:00 2001 From: Johann Date: Mon, 4 Mar 2019 16:55:10 -0800 Subject: [PATCH] add -Wmissing-prototypes clang treats -Wmissing-declarations differently than gcc. This provides similar coverage for clang. Fix vpx_clear_system_state() warning on 32bit builds: note: this declaration is not a prototype; add 'void' to make it a prototype for a zero-parameter function Change-Id: I5a424bc38d47c0a3dc751d65c1efea5733907785 --- configure | 2 ++ vpx_ports/system_state.h | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index b5096723b..2174544c3 100755 --- a/configure +++ b/configure @@ -627,6 +627,8 @@ process_toolchain() { add_cflags_only -Wframe-larger-than=52000 check_cflags -Wmissing-declarations && \ add_cflags_only -Wmissing-declarations + check_cflags -Wmissing-prototypes && \ + add_cflags_only -Wmissing-prototypes if enabled mips || [ -z "${INLINE}" ]; then enabled extra_warnings || check_add_cflags -Wno-unused-function fi diff --git a/vpx_ports/system_state.h b/vpx_ports/system_state.h index 03557e4bd..452cb5739 100644 --- a/vpx_ports/system_state.h +++ b/vpx_ports/system_state.h @@ -18,7 +18,7 @@ extern "C" { #endif #if (ARCH_X86 || ARCH_X86_64) && HAVE_MMX -extern void vpx_clear_system_state(); +extern void vpx_clear_system_state(void); #else #define vpx_clear_system_state() #endif // (ARCH_X86 || ARCH_X86_64) && HAVE_MMX -- 2.50.1