<p><a href="https://gerrit.osmocom.org/11021">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/11021/1/include/osmocom/sgsn/gprs_sgsn.h">File include/osmocom/sgsn/gprs_sgsn.h:</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/11021/1/include/osmocom/sgsn/gprs_sgsn.h@179">Patch Set #1, Line 179:</a> <code style="font-family:monospace,monospace">           int auth_reattempt; /* tracking UMTS resync auth attempts */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">yes, but it will hold only values from 0 - 2.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Then the range is that of an unsigned, and imho it's clearer just by looking at this line that no negative values are going to be used, no need to go check the code if some special values like -1 are being used.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11021">change 11021</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/11021"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I708226cec9e131dcda4234f42ed3689f4f6750e8 </div>
<div style="display:none"> Gerrit-Change-Number: 11021 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </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: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 19 Sep 2018 09:53:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>