shithub: libvpx

Info  •  Files  •  Log  •  Branches

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

--- a/configure

+++ b/configure

@@ -627,6 +627,8 @@

           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

--- a/vpx_ports/system_state.h

+++ b/vpx_ports/system_state.h

@@ -18,7 +18,7 @@

 #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