summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2019-11-08 14:23:56 +0100
committerwm4 <wm4@nowhere>2019-11-08 14:23:56 +0100
commit3e401bf652eadf39aed2dee250dfed6168760fce (patch)
treead3fab802add87b811788527514bcbd94185283e
parent3ed9c1c9701098db17a2b791f9793e75a1947cc9 (diff)
downloadmpv-3e401bf652eadf39aed2dee250dfed6168760fce.tar.bz2
mpv-3e401bf652eadf39aed2dee250dfed6168760fce.tar.xz
test: make build fail if NDEBUG is defined
Defining NDEBUG via CFLAGS is the canonical way to disable assertions in C. mpv respects this (and ta.c actually disables some debugging machinery if it's defined). But for tests, this is not useful at all. So if --enable-tests is passed to configure, the user must not define NDEBUG, even if the rest of the player does not care. (We could just #undef NDEBUG, but let's not. Tests calling into the rest of the player might depend on asserts there, or so.)
-rw-r--r--test/index.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/test/index.c b/test/index.c
index c4ad10888f..4804cead2f 100644
--- a/test/index.c
+++ b/test/index.c
@@ -51,3 +51,7 @@ bool run_tests(struct MPContext *mpctx)
return num_run > 0; // still error if none
}
+
+#ifdef NDEBUG
+static_assert(false, "don't define NDEBUG for tests");
+#endif