Skip to content

Commit c583321

Browse files
author
Francois Best
committed
feat: Increase warning level for sources
Don't do it globally as externals may have warnings we can't do much about.
1 parent 3396ec9 commit c583321

File tree

2 files changed

+18
-1
lines changed

2 files changed

+18
-1
lines changed

builder/CMakeLists.txt

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,26 @@ macro(setup_builder)
88

99
include_directories(${ROOT_SOURCE_DIR})
1010

11-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -W -Wshadow -Wunused-variable -Wunused-parameter -Wunused-function -Wunused -Wno-system-headers -Wno-deprecated -Woverloaded-virtual")
11+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
12+
-Wall \
13+
-W \
14+
-Wshadow \
15+
-Wunused-variable \
16+
-Wunused-parameter \
17+
-Wunused-function \
18+
-Wunused \
19+
-Wno-system-headers \
20+
-Wno-deprecated \
21+
-Woverloaded-virtual \
22+
")
1223
if (BUILDER_ENABLE_PROFILING)
1324
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -g -O0")
1425
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} --coverage")
1526
endif()
1627
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
1728

1829
endmacro()
30+
31+
macro(increase_warning_level)
32+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wconversion -Wsign-conversion")
33+
endmacro()

src/CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
increase_warning_level()
2+
13
project(midi)
24

35
add_library(midi STATIC

0 commit comments

Comments
 (0)