From c7365b14f31926b6fd95e0efe9e8fefbbf265d24 Mon Sep 17 00:00:00 2001 From: iphydf Date: Wed, 26 Sep 2018 00:15:52 +0000 Subject: [PATCH] Remove a function-like macro and replace it with a function. No use making this a macro. LTO will inline this anyway, if we care about performance. Generally, we avoid function-like macros that can be functions. --- toxcore/state.c | 5 +++++ toxcore/state.h | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/toxcore/state.c b/toxcore/state.c index aa01b32a09..29b4af88a3 100644 --- a/toxcore/state.c +++ b/toxcore/state.c @@ -79,6 +79,11 @@ uint16_t lendian_to_host16(uint16_t lendian) #endif } +uint16_t host_tolendian16(uint16_t host) +{ + return lendian_to_host16(host); +} + void host_to_lendian32(uint8_t *dest, uint32_t num) { #ifdef WORDS_BIGENDIAN diff --git a/toxcore/state.h b/toxcore/state.h index 154478bf6e..2d1f62c36d 100644 --- a/toxcore/state.h +++ b/toxcore/state.h @@ -39,7 +39,7 @@ uint8_t *state_write_section_header(uint8_t *data, uint16_t cookie_type, uint32_ // Utilities for state data serialisation. uint16_t lendian_to_host16(uint16_t lendian); -#define host_tolendian16(x) lendian_to_host16(x) +uint16_t host_tolendian16(uint16_t host); void host_to_lendian32(uint8_t *dest, uint32_t num); void lendian_to_host32(uint32_t *dest, const uint8_t *lendian);