Skip to content
Permalink
Browse files

Remove /WX from the CMakefile, and always undefine NDEBUG

Also, we currently don't need <assert.h> in the library

Fixes #71
  • Loading branch information...
jedisct1 committed Aug 10, 2019
1 parent 68e3c59 commit b3a2a30211bbc21998c336a0801604a1d93becec
Showing with 3 additions and 2 deletions.
  1. +0 −1 CMakeLists.txt
  2. +0 −1 impl/common.h
  3. +3 −0 tests/tests.c
@@ -78,7 +78,6 @@ set(compile_options
# ---- Optimizations ----
/Os /EHsc
# ---- Warnings ----
/WX
/W4
/wd4197 # * suppress warning "top-level volatile in cast is ignored"
/wd4146 # * suppress warning "unary minus operator applied to unsigned type, result still
@@ -1,4 +1,3 @@
#include <assert.h>
#include <errno.h>
#include <limits.h>
#include <stdbool.h>
@@ -1,3 +1,6 @@
#ifdef NDEBUG
# undef NDEBUG
#endif
#include <assert.h>
#include <stdio.h>
#include <string.h>

0 comments on commit b3a2a30

Please sign in to comment.
You can’t perform that action at this time.