This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Harald Welte gerrit-no-reply at lists.osmocom.orgHarald Welte has submitted this change and it was merged. Change subject: fix issue on big endian architecture ...................................................................... fix issue on big endian architecture OSMO_IS_LITTLE_ENDIAN is always defined and has a value of either 0 or 1 as a result in byteswap.h the corresponding swap functions will be always called, independent of the endianess Signed-off-by: Thorsten Alteholz <osmocom at alteholz.de> Change-Id: I4a09d2d8ccf155e70a3977ae1747758b6bc5125e --- M include/osmocom/core/byteswap.h 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Vadim Yanitskiy: Looks good to me, but someone else must approve Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/include/osmocom/core/byteswap.h b/include/osmocom/core/byteswap.h index 1ea5b3d..cecdc63 100644 --- a/include/osmocom/core/byteswap.h +++ b/include/osmocom/core/byteswap.h @@ -32,7 +32,7 @@ return out; } -#ifdef OSMO_IS_LITTLE_ENDIAN +#if OSMO_IS_LITTLE_ENDIAN == 1 #define osmo_ntohl(x) osmo_swab32(x) #define osmo_ntohs(x) osmo_swab16(x) #define osmo_htonl(x) osmo_swab32(x) -- To view, visit https://gerrit.osmocom.org/7666 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a09d2d8ccf155e70a3977ae1747758b6bc5125e Gerrit-PatchSet: 2 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Thorsten Alteholz <osmocom at alteholz.de> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>