<p style="white-space: pre-wrap; word-wrap: break-word;">the test is importing/building tons of modules which it doesn't need, this should be cleaned up first.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/9653">View Change</a></p><p>9 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/SCCP_Tests.ttcn">File sccp/SCCP_Tests.ttcn:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/SCCP_Tests.ttcn@37">Patch Set #1, Line 37:</a> <code style="font-family:monospace,monospace">import from BSSAP_Adapter all;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">there is nothing BSSAP specific in this test, I'm wondering why it usees BSSAP_Adapter?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/SCCP_Tests.ttcn@60">Patch Set #1, Line 60:</a> <code style="font-family:monospace,monospace">function f_init(BSSAP_Configuration cfg) runs on MTC_CT {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this appears to be the only BSSAP related type used, but we're not really using the BSSAP_Adapter.  I think this is misleading and I'd prefer if code that tests SCCP only imports no modules and/or types that relate to BSSAP</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh">File sccp/gen_links.sh:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh@49">Patch Set #1, Line 49:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">DIR=$BASEDIR/titan.ProtocolModules.BSSMAP_v11.2.0/src<br>FILES="BSSAP_Types.ttcn"<br>gen_links $DIR $FILES<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">this can go</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh@53">Patch Set #1, Line 53:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">DIR=$BASEDIR/titan.ProtocolModules.ROSE/src<br>FILES="Remote_Operations_Generic_ROS_PDUs.asn  Remote_Operations_Information_Objects.asn"<br>gen_links $DIR $FILES<br><br>DIR=$BASEDIR/titan.ProtocolModules.MAP/src<br>FILES="MAP_ApplicationContexts.asn MAP_CH_DataTypes.asn MAP_CallHandlingOperations.asn MAP_CommonDataTypes.asn MAP_DialogueInformation.asn MAP_ER_DataTypes.asn MAP_EncDec.cc MAP_Errors.asn MAP_GR_DataTypes.asn MAP_Group_Call_Operations.asn MAP_LCS_DataTypes.asn MAP_LocationServiceOperations.asn MAP_MS_DataTypes.asn MAP_MobileServiceOperations.asn MAP_OM_DataTypes.asn MAP_OperationAndMaintenanceOperations.asn MAP_PDU_Defs.asn MAP_Protocol.asn MAP_SM_DataTypes.asn MAP_SS_Code.asn MAP_SS_DataTypes.asn MAP_ShortMessageServiceOperations.asn MAP_SupplementaryServiceOperations.asn MAP_TS_Code.asn MAP_Types.ttcn "<br>FILES+="MAP_BS_Code.asn MAP_ExtensionDataTypes.asn MobileDomainDefinitions.asn"<br>gen_links $DIR $FILES<br><br>DIR=$BASEDIR/titan.ProtocolModules.MobileL3_v13.4.0/src<br>FILES="MobileL3_CC_Types.ttcn MobileL3_CommonIE_Types.ttcn MobileL3_GMM_SM_Types.ttcn MobileL3_MM_Types.ttcn MobileL3_RRM_Types.ttcn MobileL3_SMS_Types.ttcn MobileL3_SS_Types.ttcn MobileL3_Types.ttcn "<br>FILES+="SS_DataTypes.asn SS_Errors.asn SS_Operations.asn SS_PDU_Defs.asn SS_Protocol.asn SS_Types.ttcn SS_EncDec.cc"<br>gen_links $DIR $FILES<br><br>DIR=$BASEDIR/titan.ProtocolModules.SDP/src<br>FILES="SDP_EncDec.cc SDP_Types.ttcn SDP_parse_.tab.c SDP_parse_.tab.h SDP_parse_parser.h SDP_parser.l<br>SDP_parser.y lex.SDP_parse_.c"<br>gen_links $DIR $FILES<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">this can all go</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh@78">Patch Set #1, Line 78:</a> <code style="font-family:monospace,monospace">MNCC_Types.ttcn MNCC_EncDec.cc MNCC_CodecPort.ttcn mncc.h MNCC_Emulation.ttcn</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this can go</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh@79">Patch Set #1, Line 79:</a> <code style="font-family:monospace,monospace">GSUP_Types.ttcn GSUP_Emulation.ttcn</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this can go</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh@80">Patch Set #1, Line 80:</a> <code style="font-family:monospace,monospace">L3_Templates.ttcn L3_Common.ttcn</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">this can go</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/gen_links.sh@81">Patch Set #1, Line 81:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">BSSMAP_Emulation.ttcn BSSAP_CodecPort.ttcn BSSMAP_Templates.ttcn BSSAP_Adapter.ttcn MGCP_Types.ttcn MGCP_Templates.ttcn MGCP_CodecPort_CtrlFunct.ttcn MGCP_Emulation.ttcn "<br>FILES+="MGCP_CodecPort.ttcn MGCP_CodecPort_CtrlFunctDef.cc "<br>gen_links $DIR $FILES<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">this can all go</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/regen_makefile.sh">File sccp/regen_makefile.sh:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/9653/1/sccp/regen_makefile.sh@5">Patch Set #1, Line 5:</a> <code style="font-family:monospace,monospace">-DIPA_EMULATION_MGCP -DIPA_EMULATION_GSUP -DIPA_EMULATION_SCCP</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">those three should be removed, none of this is used here.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9653">change 9653</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/9653"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I03f5e8b282a7396b45417495c88d8fb81b26cda8 </div>
<div style="display:none"> Gerrit-Change-Number: 9653 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 03 Jul 2018 11:42:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>