Pau Espin Pedrol has submitted this change and it was merged. (
https://gerrit.osmocom.org/11779 )
Change subject: gsm/protocol/gsm_08_58.h: Fix typo
......................................................................
gsm/protocol/gsm_08_58.h: Fix typo
Fix a typo that prevents building on ppc64 (Power8, Big Endian).
Tested with gcc version: 8.2.1+r265487
/bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I..
-I../include -I../include -DBUILDING_LIBOSMOCORE -Wall -Wall -O2 -Wall
-D_FORTIFY_SOURCE=2 -fstack-protector-strong -funwind-tables
-fasynchronous-unwind-tables -fstack-clash-protection -g -DBUILDING_LIBOSMOCORE
-Wall -c -o panic.lo panic.c
In file included from ../include/osmocom/gsm/rsl.h:6,
from gsmtap_util.c:37:
../include/osmocom/gsm/protocol/gsm_08_58.h:60:4: error: expected
specifier-qualifier-list before 'sapi'
sapi:3;
^~~~
Change-Id: I4a40fc68cb6dfdfadfcc34213c4ecdabbbc356f2
---
M include/osmocom/gsm/protocol/gsm_08_58.h
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
Pau Espin Pedrol: Looks good to me, approved
diff --git a/include/osmocom/gsm/protocol/gsm_08_58.h
b/include/osmocom/gsm/protocol/gsm_08_58.h
index cd28caf..363f33d 100644
--- a/include/osmocom/gsm/protocol/gsm_08_58.h
+++ b/include/osmocom/gsm/protocol/gsm_08_58.h
@@ -56,7 +56,7 @@
#if OSMO_IS_BIG_ENDIAN
uint8_t cbits:2,
na:1,
- reserved:2;
+ reserved:2,
sapi:3;
#elif OSMO_IS_LITTLE_ENDIAN
uint8_t sapi:3,
--
To view, visit https://gerrit.osmocom.org/11779
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I4a40fc68cb6dfdfadfcc34213c4ecdabbbc356f2
Gerrit-Change-Number: 11779
Gerrit-PatchSet: 1
Gerrit-Owner: roox <mar...@gmx.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de>
