From 8ef48ecc99caa9c28b7d53bb45e6bd56872170a1 Mon Sep 17 00:00:00 2001 From: click Date: Tue, 27 Nov 2012 00:43:38 +0100 Subject: Core/Buildsystem: Clean up compileflags when compiling with warnings fully enabled. -Wwrite-strings is default enabled when -Wall is set in clang/gcc, hence removed. -Woverloaded-virtual is now enabled for C++-code only, this removes the useless warnings when dabbling with C-only code. --- cmake/compiler/clang/settings.cmake | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'cmake/compiler/clang') diff --git a/cmake/compiler/clang/settings.cmake b/cmake/compiler/clang/settings.cmake index add9fa8694d..ae07b1fc39b 100644 --- a/cmake/compiler/clang/settings.cmake +++ b/cmake/compiler/clang/settings.cmake @@ -2,9 +2,9 @@ add_definitions(-D_BUILD_DIRECTIVE='"$(CONFIGURATION)"') if(WITH_WARNINGS) - set(WARNING_FLAGS "-W -Wall -Wextra -Wwrite-strings -Winit-self -Woverloaded-virtual -Wfatal-errors") + set(WARNING_FLAGS "-W -Wall -Wextra -Winit-self -Wfatal-errors") set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${WARNING_FLAGS}") - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${WARNING_FLAGS}") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${WARNING_FLAGS} -Woverloaded-virtual") message(STATUS "Clang: All warnings enabled") endif() -- cgit v1.2.3