[FFmpeg-devel] [PATCH] Revert "build: error on mixed declarations and code"

Nicolas George nicolas.george at normalesup.org
Thu Dec 29 16:42:48 CET 2011


This reverts commit 9d6b2077b27980936efdc616e2f9c92a56b443fc.

This error is annoying while debugging, and if someone disables it for
convenience, it raises the odds of leaving ffmpeg unbuildable by default.

Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
---


Carl Eugen Hoyos:
> Imo, please revert and push.

I'll do that pretty soon.


Alexander Strasser:
> couldn't it be easily simulated for debugging like in:

Of course, but it takes more step for no good reason I can see. It looks as
if the people who pushed this do not trust their own developers.

Regards,

-- 
  Nicolas George


 configure |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/configure b/configure
index d0f546b..e97bcae 100755
--- a/configure
+++ b/configure
@@ -3318,7 +3318,6 @@ elif enabled gcc; then
     check_cflags -fno-tree-vectorize
     check_cflags -Werror=implicit-function-declaration
     check_cflags -Werror=missing-prototypes
-    check_cflags -Werror=declaration-after-statement
 elif enabled llvm_gcc; then
     check_cflags -mllvm -stack-alignment=16
 elif enabled clang; then
-- 
1.7.2.5



More information about the ffmpeg-devel mailing list