<p><a href="https://gerrit.osmocom.org/11504">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11504/9/src/gsm/gsm0808_utils.c">File src/gsm/gsm0808_utils.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/11504/9/src/gsm/gsm0808_utils.c@1373">Patch Set #9, Line 1373:</a> <code style="font-family:monospace,monospace">gsm0808_cell_id_u_name</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Are there any users of this function outside the library? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">oh, wait what, I thought I was adding this function.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11504/9/src/gsm/gsm23003.c">File src/gsm/gsm23003.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/11504/9/src/gsm/gsm23003.c@339">Patch Set #9, Line 339:</a> <code style="font-family:monospace,monospace">a->lac > b->lac</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">If not a < b, and not a > b, then a != b, right?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">please read your comment and the code again :)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11504/9/src/gsm/gsm23003.c@356">Patch Set #9, Line 356:</a> <code style="font-family:monospace,monospace">a->cell_identity > b->cell_identity</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">same here.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">:)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11504/9/src/gsm/libosmogsm.map">File src/gsm/libosmogsm.map:</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/11504/9/src/gsm/libosmogsm.map@213">Patch Set #9, Line 213:</a> <code style="font-family:monospace,monospace">gsm0808_cell_id_u_match</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">So, you're making this symbol static, thus breaking the API, but still keeping it here?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">gah, forgot, thx</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11504">change 11504</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/11504"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I5535f0d149c2173294538df75764dd181b023312 </div>
<div style="display:none"> Gerrit-Change-Number: 11504 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 21 Jan 2019 17:33:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>