CMakeLists.txt: enable two new compiler warnings
authorAntonio Ospite <ospite@studenti.unina.it>
Sat, 13 Jul 2013 09:05:00 +0000 (11:05 +0200)
committerAntonio Ospite <ospite@studenti.unina.it>
Sat, 20 Jul 2013 22:20:06 +0000 (00:20 +0200)
Enable -Wsign-conversion as a strict compilation check for both gcc and
clang.

For clang also enable -Wshorten-64-to-32 as a strict compilation check.

CMakeLists.txt

index d908f1c..49c3ac0 100644 (file)
@@ -78,6 +78,7 @@ if (CMAKE_COMPILER_IS_GNUCC OR CMAKE_COMPILER_IS_CLANG)
   if (STRICT_COMPILATION_CHECKS)
     add_flags(STRICT_FLAGS
       -Werror
+      -Wsign-conversion
       # NOTE: Vanilla libusb-1.0.8 can't live with -pedantic-errors
       -pedantic-errors)
 
@@ -96,6 +97,13 @@ if (CMAKE_COMPILER_IS_GNUCC)
   endif()
 endif()
 
+if (CMAKE_COMPILER_IS_CLANG)
+  if (STRICT_COMPILATION_CHECKS)
+    add_flags(STRICT_FLAGS
+      -Wshorten-64-to-32)
+  endif()
+endif()
+
 set(CMAKE_C_FLAGS_DEBUG "-O0 ${DEBUG_FLAGS} ${STRICT_FLAGS}")
 set(CMAKE_C_FLAGS_RELEASE "-O2 ${RELEASE_FLAGS} ${STRICT_FLAGS}")
 set(CMAKE_C_FLAGS_RELWITHDEBINFO "-O2 ${RELEASE_FLAGS} ${DEBUG_FLAGS} ${STRICT_FLAGS}")