On Mon, 14 Jan 2013, Diego Biurrun wrote:
These warnings have no false positives and point to serious bugs. ---Squashed both patches into one and dropped the clang options that have no effect. configure | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 3d5fe7c..751d22e 100755 --- a/configure +++ b/configure @@ -3685,6 +3685,7 @@ elif enabled gcc; then check_cflags -Werror=implicit-function-declaration check_cflags -Werror=missing-prototypes check_cflags -Werror=declaration-after-statement + check_cflags -Werror=return-type check_cflags -Werror=vla elif enabled llvm_gcc; then check_cflags -mllvm -stack-alignment=16 @@ -3693,6 +3694,7 @@ elif enabled clang; then check_cflags -Qunused-arguments check_cflags -Werror=implicit-function-declaration check_cflags -Werror=missing-prototypes + check_cflags -Werror=return-type elif enabled armcc; then # 2523: use of inline assembler is deprecated add_cflags -W${armcc_opt},--diag_suppress=2523 -- 1.7.1
Looks sensible to me // Martin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
