Skip to content

Commit

Permalink
removing -Werror and option
Browse files Browse the repository at this point in the history
  • Loading branch information
lemire committed Jan 30, 2025
1 parent 14e12df commit 4df18c9
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
all: unit bench

unit : tests/unit.c include/xorfilter.h include/binaryfusefilter.h
${CC} -std=c99 -g -O2 -fsanitize=address,leak,undefined -o unit tests/unit.c -lm -Iinclude -Wall -Wextra -Wshadow -Wcast-qual -Wconversion -Wsign-conversion -Werror
${CC} -std=c99 -g -O2 -fsanitize=address,leak,undefined -o unit tests/unit.c -lm -Iinclude -Wall -Wextra -Wshadow -Wcast-qual

ab : tests/a.c tests/b.c
${CC} -std=c99 -o c tests/a.c tests/b.c -lm -Iinclude -Wall -Wextra -Wshadow -Wcast-qual -Wconversion -Wsign-conversion
Expand Down
5 changes: 2 additions & 3 deletions tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,13 @@ target_link_libraries(unit PRIVATE xor_singleheader)


# full warnings with sanitizers for tests. Include debug symbols and
# only -O2 to maintain some debugability. -Werror to
# prevent new warning creeping in Matches Makefile
# only -O2 to maintain some debugability.
if (MSVC)
# limited support for MSVC, this is not tested
list(APPEND TEST_COMPILE_OPTIONS /W4 /fsanitize=address)
else() # *nix
list(APPEND TEST_COMPILE_OPTIONS -g -O2
-Wall -Wextra -Wshadow -Wcast-qual -Wconversion -Wsign-conversion -Werror)
-Wall -Wextra)

if (NOT MINGW) # sanitizers are not supported under mingw
list(APPEND TEST_COMPILE_OPTIONS -fsanitize=address,undefined)
Expand Down

0 comments on commit 4df18c9

Please sign in to comment.