<p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/15847">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-msc/+/15847/1/src/libmsc/ran_msg_iu.c">File src/libmsc/ran_msg_iu.c:</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-msc/+/15847/1/src/libmsc/ran_msg_iu.c@402">Patch Set #1, Line 402:</a> <code style="font-family:monospace,monospace">               LOG_RAN_IU_ENC(caller_fi, LOGL_DEBUG, "DirectTransfer\n");</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Can we then at least use Can we then use strings from value_string directly so we don't duplicate st […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">These are the only instances of these RANAP specific strings, there are no dups.<br>(The messages' names differ between RANAP and BSSMAP, using ran_msg_type_names[] would give me the BSSMAP names, but I want the RANAP names. Since BSSMAP is the "usual" mode of operation for osmo-msc, I decided to use the BSSMAP naming for the internal structures and RAN message types, but in practice these are RAN independent. Only the ran_msg_a.c and ran_msg_iu.c make the distinction between BSSMAP and RANAP encoding, so these places here where the string constants are added should be the exact only places where we want to name the individual messages according to RANAP.)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I humbly think this is a sub-tiny aspect of the upcoming patches and not worth it to discuss at length...<br>If this is still controversial, I would rather consider dropping the patch than spending more time.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I would rather get on to the interesting patches, where I really fundamentally change the way osmo-msc works ... if you allow, of course</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/15847">change 15847</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-msc/+/15847"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ib0e0630d775a28958ea86802f70cbeec07087f91 </div>
<div style="display:none"> Gerrit-Change-Number: 15847 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 29 Oct 2019 15:44:49 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>