Attention is currently required from: fixeria, neels.

pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-hnbgw/+/35645?usp=email )

Change subject: pfcp: implement sending Network Instance IEs
......................................................................


Patch Set 3:

(1 comment)

File src/osmo-hnbgw/ps_rab_fsm.c:

https://gerrit.osmocom.org/c/osmo-hnbgw/+/35645/comment/4cf59fc6_d25d9aa7
PS3, Line 244:  
set_netinst(ser->create_pdr[ser->create_pdr_count].pdi.network_inst, 
g_hnbgw->config.pfcp.netinst.core);
> (re macro: I mean, look at this, do you want to have a third one of these 
> miles long arguments here? […]
better than some macro which one for sure has to go look at when looking at the 
code because god knows what may be doing? yes.
Do as you please, not blocking.



--
To view, visit https://gerrit.osmocom.org/c/osmo-hnbgw/+/35645?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hnbgw
Gerrit-Branch: master
Gerrit-Change-Id: Ief53dbfacf1645c32a07847d590c4884d4c8ca56
Gerrit-Change-Number: 35645
Gerrit-PatchSet: 3
Gerrit-Owner: neels <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Thu, 25 Jan 2024 13:57:43 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <[email protected]>
Gerrit-MessageType: comment

Reply via email to