<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25021">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25021/3//COMMIT_MSG">Commit Message:</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/osmo-ttcn3-hacks/+/25021/3//COMMIT_MSG@1">Patch Set #3, Line 1:</a> <code style="font-family:monospace,monospace">Parent:     0882a312 (bsc: split f_start_handler() in two, ..._create() and ..._run())</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not really sure I like the fact that we send arbitrary charstrings through the COORD port, rather than some TTCN3 enum.  Using charstring doesn't really seem to provide any benefit here, or am I missing something?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Whether you add another member to the enum, or you add another charstring "const" doesn't really make a difference in terms of future extensions.  The charstring just permits the sender to send invalid values, while the enum would guarantee only one of the permitted (currently 3) values can ever be sent, otherwise a compile-time error would be generated.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25021">change 25021</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/c/osmo-ttcn3-hacks/+/25021"/><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-Change-Id: I1e7d446644f46b8a0a94688e618ffb4f23a8900e </div>
<div style="display:none"> Gerrit-Change-Number: 25021 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 27 Jul 2021 07:31:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>