EnableWarnings.cmake

Tag

Show log
Commit
Hash : 6af6170b
Author :
Date : 2019-01-05T20:50:42

Make ENABLE_WERROR actually work This change explicitly adds -Werror to the CFLAGS. Due to the way that the ADD_C_FLAG_IF_SUPPORTED() macro was mangling the flag name to convert it into a define name, any warning that had a dash in its name was not being correctly enabled. Additionally, any flag that is enabled implicitly by the compiler (like -Wunused-result and -Wdeprecated-declarations) would not cause an error unless they were explicitly enabled with the ENABLE_WARNINGS() macro.

cmake/Modules/EnableWarnings.cmake

MACRO(ENABLE_WARNINGS flag)
	ADD_C_FLAG_IF_SUPPORTED(-W${flag})
ENDMACRO()

MACRO(DISABLE_WARNINGS flag)
	ADD_C_FLAG_IF_SUPPORTED(-Wno-${flag})
ENDMACRO()

IF(ENABLE_WERROR)
	ADD_C_FLAG_IF_SUPPORTED(-Werror)
ENDIF()

Download

thodg/libgit2/cmake/Modules/EnableWarnings.cmake

Commit

cmake/Modules/EnableWarnings.cmake