diff options
author | Laurent Bercot <ska-skaware@skarnet.org> | 2017-10-18 16:13:24 +0000 |
---|---|---|
committer | Laurent Bercot <ska-skaware@skarnet.org> | 2017-10-18 16:13:24 +0000 |
commit | 45923c08b00ccdfd888caf6d2b555a137c563988 (patch) | |
tree | 5dc167cf4b8e3ff372f59ca9fb6ba0940387c83c /src/libstddjb/uint32_reverse.c | |
parent | 62a04cdd0b6811c07bbafae37fe906a36fa800b1 (diff) | |
download | skalibs-45923c08b00ccdfd888caf6d2b555a137c563988.tar.xz |
Better packing / unpacking code
Godbolted for x86_64 with gcc and clang.
Diffstat (limited to 'src/libstddjb/uint32_reverse.c')
-rw-r--r-- | src/libstddjb/uint32_reverse.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/libstddjb/uint32_reverse.c b/src/libstddjb/uint32_reverse.c index 51c9877..2d3f915 100644 --- a/src/libstddjb/uint32_reverse.c +++ b/src/libstddjb/uint32_reverse.c @@ -1,17 +1,17 @@ /* ISC license. */ +#include <stdint.h> +#include <string.h> #include <skalibs/uint32.h> void uint32_reverse (char *s, size_t n) { while (n--) { - char c = s[0] ; - s[0] = s[3] ; - s[3] = c ; - c = s[1] ; - s[1] = s[2] ; - s[2] = c ; - s += 4 ; + uint32_t x ; + memcpy(&x, s, sizeof(uint32_t)) ; + x = uint32_bswap(x) ; + memcpy(s, &x, sizeof(uint32_t)) ; + s += sizeof(uint32_t) ; } } |