[PATCH] libosmocore[master]: fix issue on big endian architecture

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/.

Thorsten Alteholz gerrit-no-reply at lists.osmocom.org
Sat Apr 7 21:11:02 UTC 2018


Review at  https://gerrit.osmocom.org/7666

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(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/66/7666/1

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: newchange
Gerrit-Change-Id: I4a09d2d8ccf155e70a3977ae1747758b6bc5125e
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Thorsten Alteholz <osmocom at alteholz.de>



More information about the gerrit-log mailing list