Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39849?usp=email )
Change subject: s1gw: derive altsteps from f_ConnHdlr_rx_s1ap_from_{enb,mme}() ......................................................................
Patch Set 1:
(2 comments)
File s1gw/S1GW_ConnHdlr.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39849/comment/98b29898_95c2f... : PS1, Line 154: [] as_ConnHdlr_s1ap_from_enb(pdu, tr_pdu) {
Not sure what do you win with this but OK....
The altstep name clearly indicates direction from where the PDU is coming from (from eNB). With the old naming this was less obvious.
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39849/comment/f9ea8bb8_a47c5... : PS1, Line 187: as_ConnHdlr_s1ap_from_mme Here we eliminate the need to use `S1AP_RecvFrom` and get `S1AP_PDU` directly.