diff --git a/CMakeLists.txt b/CMakeLists.txt index c630da1..aa0d3e0 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -111,6 +111,20 @@ target_link_libraries (degesch ${project_libraries}) add_executable (kike kike.c kike-replies.c ${common_sources} ${common_headers}) target_link_libraries (kike ${project_libraries}) +# Various clang-based diagnostics, loads of fake positives and spam +file (GLOB clang_tidy_sources *.c) +set (clang_tidy_checks misc-* readability-* + -readability-braces-around-statements + -readability-named-parameter) +string (REPLACE ";" "," clang_tidy_checks "${clang_tidy_checks}") + +set (CMAKE_EXPORT_COMPILE_COMMANDS ON) +add_custom_target (clang-tidy + COMMAND clang-tidy -p ${PROJECT_BINARY_DIR} -checks=${clang_tidy_checks} + ${clang_tidy_sources} 1>&2 + USES_TERMINAL + WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}) + # Installation install (TARGETS zyklonb degesch kike DESTINATION ${CMAKE_INSTALL_BINDIR}) install (FILES LICENSE DESTINATION ${CMAKE_INSTALL_DOCDIR}) diff --git a/degesch.c b/degesch.c index 543601e..9276b8b 100644 --- a/degesch.c +++ b/degesch.c @@ -1868,7 +1868,7 @@ serialize_configuration (struct app_context *ctx, struct str *output) /// Builds a color pair for 256-color terminals with a 16-color backup value #define COLOR_256(name, c256) \ - (((COLOR_ ## name) & 0xFFFF) | ((c256 & 0xFFFF) << 16)) + (((COLOR_ ## name) & 0xFFFF) | (((c256) & 0xFFFF) << 16)) static struct { @@ -4288,6 +4288,7 @@ transport_tls_init (struct server *s, struct error **e) // XXX: is this a reason to abort the connection? log_server_error (s, s->buffer, "#s", error->message); error_free (error); + error = NULL; } SSL_set_connect_state (ssl);