dexter has posted comments on this change. ( https://gerrit.osmocom.org/11690 )
Change subject: MSC_Tests: remove unused control interface ...................................................................... Patch Set 7: (1 comment) > I mean, it would be nicer to allow having a ctrl for both sides, > but if it fixes the status quo, this patch is ok, right? Making the control interface available from both sides requires an additional architecture to route the message back and forth between the two entities. The patch does not break anything, also having the control interface available at the BSC_ConnectionHandler makes more sense than having it on the MTC_CT. https://gerrit.osmocom.org/#/c/11690/4/msc/MSC_Tests.ttcn File msc/MSC_Tests.ttcn: https://gerrit.osmocom.org/#/c/11690/4/msc/MSC_Tests.ttcn@a20 PS4, Line 20: > You remove those 2 here and introduce them in next patch. […] Done -- To view, visit https://gerrit.osmocom.org/11690 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I204b2e96057d13342f4bd4fdaf08fb7b88b6b11d Gerrit-Change-Number: 11690 Gerrit-PatchSet: 7 Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder (1000002) Gerrit-Reviewer: Max <[email protected]> Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-Reviewer: dexter <[email protected]> Gerrit-Comment-Date: Fri, 07 Dec 2018 14:27:54 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No
