Skip to content

Commit 35b746f

Browse files
committed
Enabled more compiler warnings
Minor change Minor change
1 parent 80e53ef commit 35b746f

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,11 +221,11 @@ check_compiler()
221221

222222
if(DEFINED ENV{WERROR} AND "$ENV{WERROR}")
223223
message(STATUS "Will compile with -Werror. ")
224-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wformat-security -Werror -Wno-sign-compare -Wno-reorder -Wno-unused-variable -Wno-unused-but-set-variable -Wfatal-errors")
224+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wno-non-virtual-dtor -Wno-long-long -Wwrite-strings -Wpointer-arith -Woverloaded-virtual -Wno-variadic-macros -Wshadow -Wformat-security -Werror -Wno-sign-compare -Wno-reorder -Wno-unused-variable -Wno-unused-but-set-variable -Wfatal-errors")
225225
else()
226226
if (NOT APPLE)
227227
message(STATUS "Set env WERROR to 1 to enable -Werror. If origin/dev compiles on your platform with that option, it is definitly a good idea to do that.")
228-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wformat-security -Wno-sign-compare -Wno-reorder -Wno-unused-variable -Wno-unused-but-set-variable")
228+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -pedantic -Wno-non-virtual-dtor -Wno-long-long -Wwrite-strings -Wpointer-arith -Woverloaded-virtual -Wno-variadic-macros -Wshadow -Wformat-security -Wno-sign-compare -Wno-reorder -Wno-unused-variable -Wno-unused-but-set-variable")
229229
endif()
230230
endif()
231231
set(CMAKE_CXX_FLAGS "-D_GLIBCXX_ASSERTIONS ${CMAKE_CXX_FLAGS}" )

los/calib/R3BLosCal2Hit.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,6 @@ class R3BLosCal2Hit : public FairTask
244244
TH2F* fhQ3_vs_Q7_corr;
245245
TH2F* fhQ4_vs_Q8;
246246
TH2F* fhQ4_vs_Q8_corr;
247-
;
248247
TH2F* fhTresX_M;
249248
TH2F* fhTresY_M;
250249
TH2F* fhTresX_T;

0 commit comments

Comments
 (0)