Attention is currently required from: fixeria.

neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ggsn/+/31511 )

Change subject: gtp: use OSMO_ASSERT() in gtp_new()
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

File gtp/gsn.c:

https://gerrit.osmocom.org/c/osmo-ggsn/+/31511/comment/b9878e02_215a246b
PS2, Line 444:  OSMO_ASSERT(gtp_T_defs[0].default_val != 0);
interesting, default_val is defined const, maybe this is a compiler bug



--
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/31511
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Ia8af1736b63d501661046fe70befe5bbabc1045a
Gerrit-Change-Number: 31511
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Sun, 26 Feb 2023 22:11:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Reply via email to