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 /include | |
| 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 'include')
| -rw-r--r-- | include/erasurecode/erasurecode_helpers_ext.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/erasurecode/erasurecode_helpers_ext.h b/include/erasurecode/erasurecode_helpers_ext.h index 740d7db..8041a57 100644 --- a/include/erasurecode/erasurecode_helpers_ext.h +++ b/include/erasurecode/erasurecode_helpers_ext.h @@ -72,8 +72,6 @@ int set_backend_id(char *buf, ec_backend_id_t id); int get_backend_id(char *buf, ec_backend_id_t *id); int set_backend_version(char *buf, uint32_t version); int get_backend_version(char *buf, uint32_t *version); -int set_metadata_chksum(char *buf); -uint32_t *get_metadata_chksum(char *buf); int is_invalid_fragment_header(fragment_header_t *header); /* ==~=*=~==~=*=~==~=*=~==~=*=~==~=*=~==~=*=~==~=*=~==~=*=~==~=*=~==~=*=~== */ |