diff options
author | Tim Burke <tim.burke@gmail.com> | 2017-04-21 18:32:02 -0700 |
---|---|---|
committer | Tim Burke <tim.burke@gmail.com> | 2017-07-06 18:02:02 +0000 |
commit | e426aee95b616778e4a8e484c3a56c691a2b9c52 (patch) | |
tree | d8e7b9870a54c7e88c7c6d76a3fdfe4b6f2c865e /test | |
parent | a9b20ae6a38073afe91ae2b7d789ddfb7dabade8 (diff) | |
download | liberasurecode-e426aee95b616778e4a8e484c3a56c691a2b9c52.tar.gz |
Un-inline get/set_metatdata_chksum
Each was only really used in one place, they had some strange return types,
and recent versions of clang on OS X would refuse to compile with
erasurecode_helpers.c:531:26: error: taking address of packed member 'metadata_chksum' of
class or structure 'fragment_header_s' may result in an unaligned pointer value
[-Werror,-Waddress-of-packed-member]
return (uint32_t *) &header->metadata_chksum;
^~~~~~~~~~~~~~~~~~~~~~~
We don't really *care* about the pointer; we just want the value!
Change-Id: I8a5e42312948a75f5dd8b23b6f5ccfa7bd22eb1d
Diffstat (limited to 'test')
-rw-r--r-- | test/liberasurecode_test.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/test/liberasurecode_test.c b/test/liberasurecode_test.c index 68c1c13..e62df94 100644 --- a/test/liberasurecode_test.c +++ b/test/liberasurecode_test.c @@ -1030,15 +1030,11 @@ static void encode_decode_test_impl(const ec_backend_id_t be_id, int cmp_size = -1; char *data_ptr = NULL; char *frag = NULL; - uint32_t *mcksum = NULL; frag = (i < args->k) ? encoded_data[i] : encoded_parity[i - args->k]; assert(frag != NULL); fragment_header_t *header = (fragment_header_t*)frag; assert(header != NULL); - mcksum = get_metadata_chksum(frag); - assert(mcksum != NULL); - assert(header->metadata_chksum == *mcksum); fragment_metadata_t metadata = header->meta; assert(metadata.idx == i); |