fix: use more strict compiler flags

Author: anonrig

BUILD.bazel

@@ -17,6 +17,12 @@ cc_library(
     name = "ncrypto",
     srcs = glob(["src/*.cpp"]),
     hdrs = glob(["include/*.h", "include/ncrypto/*.h"]),
+    copts = [
+        "-Werror",
+        "-Wextra",
+        "-Wno-unused-parameter",
+        "-Wimplicit-fallthrough",
+    ],
     includes = ["include"],
     local_defines = {
         "NCRYPTO_BSSL_LIBDECREPIT_MISSING": select(

cmake/ncrypto-flags.cmake

@@ -9,6 +9,8 @@ set(CMAKE_CXX_STANDARD 20)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 
+
+
 find_program(CCACHE_FOUND ccache)
 if(CCACHE_FOUND)
   message(STATUS "Ccache found using it as compiler launcher.")

include/ncrypto/aead.h

@@ -19,7 +19,7 @@ class Aead final {
 
  public:
   Aead() = default;
-  Aead(const AeadInfo* info, const EVP_AEAD* aead) : info_(info), aead_(aead) {}
+  Aead(const AeadInfo* info, const EVP_AEAD* aead) : aead_(aead), info_(info) {}
   Aead(const Aead&) = default;
   Aead& operator=(const Aead&) = default;
   NCRYPTO_DISALLOW_MOVE(Aead)

src/CMakeLists.txt

@@ -1,4 +1,13 @@
 add_library(ncrypto ncrypto.cpp engine.cpp aead.cpp)
+
+# Enable strict warning flags for ncrypto sources only
+if(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang|AppleClang")
+    target_compile_options(ncrypto PRIVATE -Werror -Wextra -Wno-unused-parameter -Wimplicit-fallthrough)
+endif()
+if(MSVC)
+    target_compile_options(ncrypto PRIVATE /WX /W4)
+endif()
+
 if (NCRYPTO_SHARED_LIBS)
     target_link_libraries(ncrypto PUBLIC OpenSSL::SSL OpenSSL::Crypto)
 else()