Skip to content

Commit d75dc59

Browse files
Merge #1333: test: Warn if both VERIFY and COVERAGE are defined
e83801f test: Warn if both `VERIFY` and `COVERAGE` are defined (Hennadii Stepanov) Pull request description: Solves one item in #1235. Also see: #1113 (comment). ACKs for top commit: sipa: utACK e83801f real-or-random: ACK e83801f Tree-SHA512: 25e10a09ba2c3585148becd06f2a03d85306208bda333827c9ba73eb7fd94ad15536f10daf1b335703e5cb0539584f001501ce9c578f478ff1ebc1051aefde7d
2 parents 908e02d + e83801f commit d75dc59

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/tests.c

+3
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@
1414
#pragma message("Ignoring USE_EXTERNAL_CALLBACKS in tests.")
1515
#undef USE_EXTERNAL_DEFAULT_CALLBACKS
1616
#endif
17+
#if defined(VERIFY) && defined(COVERAGE)
18+
#pragma message("Defining VERIFY for tests being built for coverage analysis support is meaningless.")
19+
#endif
1720
#include "secp256k1.c"
1821

1922
#include "../include/secp256k1.h"

0 commit comments

Comments
 (0)