laforge has submitted this change. (
https://gerrit.osmocom.org/c/libosmocore/+/33160 )
Change subject: gsm48_rest_octets: fix wrong value
......................................................................
gsm48_rest_octets: fix wrong value
The compiler complains:
gsm48_rest_octets.c:79:11: warning: initializer overrides prior
initialization of this subobject [-Winitializer-overrides]
79 | [0x08] = {2, 2},
| ^~~~~~~
gsm48_rest_octets.c:78:11: note: previous initialization is here
78 | [0x08] = {2, 1},
| ^~~~~~~
And a quick look at 04.18 confirms:
0 1 0 0 0 2 1
0 1 0 0 1 2 2
Change-Id: I672e2ce53fd07157d15fec350cfbe32a19a52941
---
M src/gsm/gsm48_rest_octets.c
1 file changed, 25 insertions(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
fixeria: Looks good to me, approved
diff --git a/src/gsm/gsm48_rest_octets.c b/src/gsm/gsm48_rest_octets.c
index b63c63f..57c2c99 100644
--- a/src/gsm/gsm48_rest_octets.c
+++ b/src/gsm/gsm48_rest_octets.c
@@ -76,7 +76,7 @@
[0x07] = {2, 0},
[0x08] = {2, 1},
- [0x08] = {2, 2},
+ [0x09] = {2, 2},
[0x0a] = {2, 3},
[0x0b] = {2, 4},
[0x0c] = {2, 5},
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/33160
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I672e2ce53fd07157d15fec350cfbe32a19a52941
Gerrit-Change-Number: 33160
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged