mpusz/mp-units
1
0
Fork 1
mirror of https://github.com/mpusz/mp-units.git synced 2025-06-25 01:01:33 +02:00
Code Issues Packages Projects Releases Wiki Activity

build: new compilation warnings enabled

Browse Source 
Refers to #385
This commit is contained in:
Mateusz Pusz
2022-08-10 12:42:43 +02:00
parent 02bf37abc4
commit e0d5e25104
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
cmake/warnings.cmake
View File

@ -68,6 +68,7 @@ macro(_set_flags)
-Wsign-conversion # warn on sign conversions -Wsign-conversion # warn on sign conversions
-Wnull-dereference # warn if a null dereference is detected -Wnull-dereference # warn if a null dereference is detected
-Wformat=2 # warn on security issues around functions that format output (ie printf) -Wformat=2 # warn on security issues around functions that format output (ie printf)
-Wdangling-else # warn about constructions where there may be confusion to which if statement an else branch belongs
) )
set(CLANG_WARNINGS ${GCC_COMMON_WARNINGS}) set(CLANG_WARNINGS ${GCC_COMMON_WARNINGS})
@ -79,6 +80,8 @@ macro(_set_flags)
-Wduplicated-cond # warn if if / else chain has duplicated conditions -Wduplicated-cond # warn if if / else chain has duplicated conditions
-Wduplicated-branches # warn if if / else branches have duplicated code -Wduplicated-branches # warn if if / else branches have duplicated code
-Wlogical-op # warn about logical operations being used where bitwise were probably wanted -Wlogical-op # warn about logical operations being used where bitwise were probably wanted
-Wduplicated-branches # warn when an if-else has identical branches
-Wduplicated-cond # warn about duplicated conditions in an if-else-if chain
) )
if(${projectPrefix}WARNINGS_AS_ERRORS) if(${projectPrefix}WARNINGS_AS_ERRORS)
@ -114,6 +117,7 @@ endfunction()
# Set compiler warning level for a provided CMake target # Set compiler warning level for a provided CMake target
function(set_target_warnings target scope) function(set_target_warnings target scope)
set(scopes PUBLIC INTERFACE PRIVATE) set(scopes PUBLIC INTERFACE PRIVATE)
if(NOT scope IN_LIST scopes) if(NOT scope IN_LIST scopes)
message(FATAL_ERROR "'scope' argument should be one of ${scopes} ('${scope}' received)") message(FATAL_ERROR "'scope' argument should be one of ${scopes} ('${scope}' received)")
endif() endif()