<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/19049">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-trx/+/19049/1/CommonLibs/trx_rate_ctr.cpp">File CommonLibs/trx_rate_ctr.cpp:</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-trx/+/19049/1/CommonLibs/trx_rate_ctr.cpp@263">Patch Set #1, Line 263:</a> <code style="font-family:monospace,monospace"> threshold_timer_sched_secs = OSMO_MAX((int)(min_secs / 2 - 1), 1);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think you need to cast 'min_secs' before the operation, otherwise you're still getting an unsigned int cast back to an int. That can go well if both are of the same size, but code shouldn't depend on size_t being identical size as int.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  ((int)min_secs) / 2 - 1</pre><p style="white-space: pre-wrap; word-wrap: break-word;">And then you're assigning it back to an unsigned int??</p><p style="white-space: pre-wrap; word-wrap: break-word;">or maybe I'm not understanding the issue...</p><p style="white-space: pre-wrap; word-wrap: break-word;">"for 5 underruns the statement in OSMO_MAX becomes -1"<br>wondering how 5 / 2 - 1 can ever become -1...</p><p style="white-space: pre-wrap; word-wrap: break-word;">Can you explain for dummies in the commit log? Can't wrap my head around it</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/19049">change 19049</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-trx/+/19049"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I6ea3d64dff189a5bc924e72d846e02d50536a8ea </div>
<div style="display:none"> Gerrit-Change-Number: 19049 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 29 Jun 2020 16:13:55 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>