Commit f5fbf9f1 authored by Alberto Miranda's avatar Alberto Miranda ♨️
Browse files

Merge branch '42-enable-werror-build-flag' into 'main' 

Resolve "Enable `-Werror` build flag"

Closes #42

See merge request !29
parents 721f8224 0c215ef2

CMakeLists.txt

+1 −1
Original line number Diff line number Diff line
@@ -293,7 +293,7 @@ find_package(RedisPlusPlus 1.3.3 REQUIRED) 
# ##############################################################################



# set compile flags

add_compile_options("-Wall" "-Wextra" "$<$<CONFIG:RELEASE>:-O3>")

add_compile_options("-Wall" "-Wextra" "-Werror" "$<$<CONFIG:RELEASE>:-O3>")

if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang")

  add_compile_options("-stdlib=libc++")

else ()

src/scord-ctl/rpc_handlers.cpp

+1 −1
Original line number Diff line number Diff line
@@ -29,7 +29,7 @@ 
static void

ADM_ping(hg_handle_t h) {



    hg_return_t ret;

    [[maybe_unused]] hg_return_t ret;



    [[maybe_unused]] margo_instance_id mid = margo_hg_handle_get_instance(h);