shithub: puzzles

Download patch

ref: b107decdaf7a49e3cc4b6f6f37c004cd8e2b5119
parent: b591bbdb5f1588f34bb07729057101d4f8b6907a
author: Ben Harris <[email protected]>
date: Sat Feb 18 08:52:18 EST 2023

Use -Wmissing-prototypes with GCC as well

-Wmissing-prototypes was what I wanted all along, but somehow I'd
mis-read the documentation and thought it wasn't.

--- a/cmake/platforms/unix.cmake
+++ b/cmake/platforms/unix.cmake
@@ -54,16 +54,11 @@
 
 if(STRICT AND (CMAKE_C_COMPILER_ID MATCHES "GNU" OR
                CMAKE_C_COMPILER_ID MATCHES "Clang"))
-  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wall -Wwrite-strings -std=c99 -pedantic -Werror")
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wall -Wwrite-strings -Wmissing-prototypes -std=c99 -pedantic -Werror")
 endif()
 
-if(STRICT AND (CMAKE_C_COMPILER_ID MATCHES "GNU"))
-  # -Wmissing-declarations is spelled differently in Clang.
-  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wmissing-declarations")
-endif()
-
 if(STRICT AND (CMAKE_C_COMPILER_ID MATCHES "Clang"))
-  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wmissing-prototypes -Wmissing-variable-declarations")
+  set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wmissing-variable-declarations")
 endif()
 
 add_compile_definitions(HELP_DIR="${CMAKE_INSTALL_PREFIX}/share/sgt-puzzles/help")