Attention is currently required from: daniel, fixeria, lynxis lazus, pespin.

osmith has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34255?usp=email )

Change subject: dia2gsup: Introduce new error scenario tests
......................................................................


Patch Set 2:

(2 comments)

File dia2gsup/DIA2GSUP_Tests.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34255/comment/38bebbb7_1a89220f
PS2, Line 249: 2
add the GSUP cause enum to ttcn3 as well?


https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34255/comment/4cdc95a8_32886c0b
PS2, Line 266:  var D2G_ConnHdlr vc_conn;
IMHO new lines between var blocks and f_init() would make this more readable



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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ic870a2c1de74bee3f7d8e26874635d58469ccbea
Gerrit-Change-Number: 34255
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: lynxis lazus <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Attention: daniel <[email protected]>
Gerrit-Attention: lynxis lazus <[email protected]>
Gerrit-Comment-Date: Wed, 06 Sep 2023 11:57:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to