Skip to content

[Build]: Enable warnings-as-errors - #6

Merged
nircoe merged 2 commits into
mainfrom
nircoe/main/werror
Sep 27, 2026
Merged

nircoe merged 2 commits into
mainfrom
nircoe/main/werror

Conversation

@nircoe

@nircoe nircoe commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

logcoe's targets built with no compiler warning flags.
They now build with -Werror -Wall -Wextra -Wpedantic on GCC/Clang and /W4 /WX on MSVC.

  • Add enable_strict_warnings to cmake/utils.cmake and apply it to logcoe, logcoe_tests, and logcoe_strip_tests
  • Drop unused parameter names from the NDEBUG stub definitions in src/logcoe.cpp
  • Remove unused this captures from thread-test lambdas

@nircoe nircoe self-assigned this Sep 27, 2026
@nircoe
nircoe marked this pull request as ready for review September 27, 2026 09:27
@nircoe
nircoe merged commit 8fd5373 into main Sep 27, 2026
8 checks passed
@nircoe
nircoe deleted the nircoe/main/werror branch September 27, 2026 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant