Attention is currently required from: laforge, lynxis lazus, pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/35625?usp=email )
Change subject: gsm_04_08_gprs.h: Introduce packed struct gsm48_pdp_address
......................................................................
Patch Set 2: Code-Review+1
(2 comments)
File include/osmocom/gsm/protocol/gsm_04_08_gprs.h:
https://gerrit.osmocom.org/c/libosmocore/+/35625/comment/96b1bef5_eba53381
PS1, Line 316: uint8_t organization:4, /* enum gsm48_pdp_type_org */
Ack
Not addressed, but looks like it's `struct_endianness.py` is forcing this formatting?
https://gerrit.osmocom.org/c/libosmocore/+/35625/comment/97de1324_eac812b9
PS1, Line 340: ietf
Yeah you cannot do that, but I don't see it being a problem. […]
Ack, just mentioning. Not critical.
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/35625?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I453ced42dd36df3a6ddb5db67093df5a22cad18c
Gerrit-Change-Number: 35625
Gerrit-PatchSet: 2
Gerrit-Owner: pespin
pespin@sysmocom.de
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria
vyanitskiy@sysmocom.de
Gerrit-Reviewer: laforge
laforge@osmocom.org
Gerrit-Reviewer: lynxis lazus
lynxis@fe80.eu
Gerrit-Attention: laforge
laforge@osmocom.org
Gerrit-Attention: pespin
pespin@sysmocom.de
Gerrit-Attention: lynxis lazus
lynxis@fe80.eu
Gerrit-Comment-Date: Fri, 19 Jan 2024 23:35:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin
pespin@sysmocom.de
Comment-In-Reply-To: fixeria
vyanitskiy@sysmocom.de
Gerrit-MessageType: comment