[CinCV-commits] CinelerraCV.git (#1228) - master (branch) updated: v2.1.5-185-gd83b8d5

git version control git at cinelerra-cv.org
Thu Jun 26 17:50:08 CEST 2014


The branch, master has been updated
       via  d83b8d51d2d2c691dccd4b3120c4d0197ed47e5c (commit)
      from  38d0d2aa5b4d3534596199f44092acdf9a02ab79 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d83b8d51d2d2c691dccd4b3120c4d0197ed47e5c
Author: Petter Reinholdtsen <pere at hungry.com>
Date:   Thu Jun 26 17:49:32 2014 +0200

    Tune warning level for C and C++ code.
    
    The extra compiler flags added to the C++ builds (-Wformat and
    -Werror=format-security) were by mistake left out from the C builds.
    Add them there.
    
    Also, ensure that to compiler flags used with configure flag
    --with-lots-of-warnings include the -Werror=format-security flag to fail
    the same way as the normal build if a security related format error is
    detected, and use more flags also for C builds, not only C++ builds.

-----------------------------------------------------------------------

Summary of changes:
 configure.ac |   32 ++++++++++++++++++++++++++++++++
 1 files changed, 32 insertions(+), 0 deletions(-)

The diff of changes is:
diff --git a/configure.ac b/configure.ac
index 0280130..f6b69b4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -30,6 +30,19 @@ AC_DEFUN([GREP_CXXFLAGS],
   ;;
 esac
 ])
+dnl
+dnl GREP_CFLAGS(flag, cmd_if_missing, cmd_if_present)
+dnl
+AC_DEFUN([GREP_CFLAGS],
+[case "$CFLAGS" in
+"$1" | "$1 "* | *" $1" | *" $1 "* )
+  ifelse($#, 3, [$3], [:])
+  ;;
+*)
+  $2
+  ;;
+esac
+])
 
 if eval "test x$GCC = xyes"; then
   for flag in \
@@ -38,10 +51,17 @@ if eval "test x$GCC = xyes"; then
   do
     GREP_CXXFLAGS($flag, [ CXXFLAGS="$CXXFLAGS $flag" ])
   done
+  for flag in \
+      -Wformat \
+      -Werror=format-security
+  do
+    GREP_CFLAGS($flag, [ CFLAGS="$CFLAGS $flag" ])
+  done
   AC_ARG_WITH([lots-of-warnings],
 	AC_HELP_STRING([--with-lots-of-warnings], [ask the compiler to enable lots of warning flags]),[
   for flag in \
       -Wall \
+      -Werror=format-security \
       -Wextra \
       -Wcast-qual \
       -Wpointer-arith \
@@ -51,6 +71,18 @@ if eval "test x$GCC = xyes"; then
   do
     GREP_CXXFLAGS($flag, [ CXXFLAGS="$CXXFLAGS $flag" ])
   done
+  for flag in \
+      -Wall \
+      -Werror=format-security \
+      -Wextra \
+      -Wcast-qual \
+      -Wpointer-arith \
+      -Wreturn-type \
+      -Wunused \
+      -pedantic
+  do
+    GREP_CFLAGS($flag, [ CFLAGS="$CFLAGS $flag" ])
+  done
 ])
 fi
 


hooks/post-receive
-- 
CinelerraCV.git (CinelerraCV)

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CinelerraCV.git" (CinelerraCV).



More information about the cinelerra-commits mailing list