Skip to content

Commit 8d0eda0

Browse files
testrand: Remove testrand_finish
This removes printing of the "random run = " at the end of the tests. I haven't seen a single case where this proved to be useful. And as of 48789da, this is anyway printed only at the end of the exhaustive tests and not the normal tests, so the probability that this will be useful in the future is very low.
1 parent 95e6815 commit 8d0eda0

3 files changed

Lines changed: 0 additions & 11 deletions

File tree

src/testrand.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,4 @@ static void testrand_flip(unsigned char *b, size_t len);
4242
/** Initialize the test RNG using (hex encoded) array up to 16 bytes, or randomly if hexseed is NULL. */
4343
static void testrand_init(const char* hexseed);
4444

45-
/** Print final test information. */
46-
static void testrand_finish(void);
47-
4845
#endif /* SECP256K1_TESTRAND_H */

src/testrand_impl.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -158,10 +158,4 @@ static void testrand_init(const char* hexseed) {
158158
testrand_seed(seed16);
159159
}
160160

161-
static void testrand_finish(void) {
162-
unsigned char run32[32];
163-
testrand256(run32);
164-
printf("random run = %02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x%02x\n", run32[0], run32[1], run32[2], run32[3], run32[4], run32[5], run32[6], run32[7], run32[8], run32[9], run32[10], run32[11], run32[12], run32[13], run32[14], run32[15]);
165-
}
166-
167161
#endif /* SECP256K1_TESTRAND_IMPL_H */

src/tests_exhaustive.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -459,8 +459,6 @@ int main(int argc, char** argv) {
459459
secp256k1_context_destroy(ctx);
460460
}
461461

462-
testrand_finish();
463-
464462
printf("no problems found\n");
465463
return EXIT_SUCCESS;
466464
}

0 commit comments

Comments
 (0)