Skip to content

Commit

Permalink
feat: Add -Wmissing-declarations for GCC
Browse files Browse the repository at this point in the history
As we start to add free standing functions, we should make
sure that they're either `static` or have a declaration in
some (public) header.
  • Loading branch information
ChristianTackeGSI committed Dec 22, 2023
1 parent 3f50d57 commit d0f6de5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmake/modules/CheckCompiler.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ if (CMAKE_SYSTEM_NAME MATCHES Linux)

# Configure build types
set(CMAKE_CONFIGURATION_TYPES "RELWITHDEBINFO" "NIGHTLY" "TEST" "RELEASE" "DEBUG" "DEBUGFULL" "PROFILE" "ARRAY_CHECK")
set(_FR_COMMON_CXX_FLAGS "-Wshadow -Wall")
set(_FR_COMMON_CXX_FLAGS "-Wshadow -Wall -Wmissing-declarations")

set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "${_FR_COMMON_CXX_FLAGS} -O2 -g")
set(CMAKE_CXX_FLAGS_NIGHTLY "${_FR_COMMON_CXX_FLAGS} -O0 -g")
Expand Down

0 comments on commit d0f6de5

Please sign in to comment.