Skip to content

Commit

Permalink
Merge #1333: test: Warn if both VERIFY and COVERAGE are defined
Browse files Browse the repository at this point in the history
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
  • Loading branch information
real-or-random committed May 31, 2023
2 parents 908e02d + e83801f commit d75dc59
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/tests.c
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@
#pragma message("Ignoring USE_EXTERNAL_CALLBACKS in tests.")
#undef USE_EXTERNAL_DEFAULT_CALLBACKS
#endif
#if defined(VERIFY) && defined(COVERAGE)
#pragma message("Defining VERIFY for tests being built for coverage analysis support is meaningless.")
#endif
#include "secp256k1.c"

#include "../include/secp256k1.h"
Expand Down

0 comments on commit d75dc59

Please sign in to comment.