diff options
author | David Robillard <d@drobilla.net> | 2022-11-25 09:24:18 -0500 |
---|---|---|
committer | David Robillard <d@drobilla.net> | 2022-11-25 10:08:49 -0500 |
commit | ec3979f6ec67ae2a2240c01763fd0d7ca0aff96b (patch) | |
tree | 62ccb5e15fcd9360090be4055db9358700c183e4 /benchmark | |
parent | b2b54b8a11d96f934066af7b5b7d1bce2dd10c57 (diff) | |
download | zix-ec3979f6ec67ae2a2240c01763fd0d7ca0aff96b.tar.gz zix-ec3979f6ec67ae2a2240c01763fd0d7ca0aff96b.tar.bz2 zix-ec3979f6ec67ae2a2240c01763fd0d7ca0aff96b.zip |
Avoid volatile side-effect within assert
Diffstat (limited to 'benchmark')
-rw-r--r-- | benchmark/dict_bench.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/benchmark/dict_bench.c b/benchmark/dict_bench.c index 85a4755..bf8a91a 100644 --- a/benchmark/dict_bench.c +++ b/benchmark/dict_bench.c @@ -167,7 +167,11 @@ run(FILE* const fd) char* volatile match = (char*)g_hash_table_lookup(hash, inputs.chunks[index].buf); - assert(!strcmp(match, inputs.chunks[index].buf)); +#ifndef NDEBUG + const char* const m = match; + assert(!strcmp(m, inputs.chunks[index].buf)); +#endif + (void)match; } fprintf(search_dat, "\t%lf", bench_end(&search_start)); @@ -179,8 +183,12 @@ run(FILE* const fd) const ZixChunk* volatile match = (const ZixChunk*)zix_hash_find_record(zhash, &inputs.chunks[index]); - assert(match); - assert(!strcmp(match->buf, inputs.chunks[index].buf)); +#ifndef NDEBUG + const ZixChunk* const m = match; + assert(m); + assert(!strcmp(m->buf, inputs.chunks[index].buf)); +#endif + (void)match; } fprintf(search_dat, "\t%lf\n", bench_end(&search_start)); |