<p style="white-space: pre-wrap; word-wrap: break-word;">This change is ready for review.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/18506">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/libosmocore/+/18506/2/src/gsm/gsm23236.c">File src/gsm/gsm23236.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/libosmocore/+/18506/2/src/gsm/gsm23236.c@36">Patch Set #2, Line 36:</a> <code style="font-family:monospace,monospace">int16_t</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Why is it signed? Can the NRI be negative?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">In the end, no real reason, just "why not".</p><p style="white-space: pre-wrap; word-wrap: break-word;">The NRI is normally <= 10 bit wide, as per 3GPP TS 23.236. (I'm going to allow up to 15 bits, because there is no hard limit up to 24 bits, but 15 is also far above what anyone would likely ever use.)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I want to use negative NRI to indicate invalid or that none has been set.<br>Come to think of it, I think I'm not actually using that anywhere now that these functions return an error code instead of an NRI, but it seems a good idea to keep some room in the definition from the start, since we are getting it for free.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The easiest is to take <0 meaning invalid, given the ample unused number space.</p><p style="white-space: pre-wrap; word-wrap: break-word;">We could also use 'int', actually. Just thought the int16_t might be more specific on how little of the int will really be used.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/18506">change 18506</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/libosmocore/+/18506"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Icb57a2dd9323c7ea11b34003eccc7e68a0247bf5 </div>
<div style="display:none"> Gerrit-Change-Number: 18506 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: Vadim Yanitskiy <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Jun 2020 15:23:22 +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: Vadim Yanitskiy <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>