<p><a href="https://gerrit.osmocom.org/c/osmo-pcu/+/15425">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15425/2/src/bts.cpp">File src/bts.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/15425/2/src/bts.cpp@595">Patch Set #2, Line 595:</a> <code style="font-family:monospace,monospace">-2002</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Looks odd to me, could you please add a comment? Why exactly -2002?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Because I had to pick a number for the timer ;)</p><p style="white-space: pre-wrap; word-wrap: break-word;">We usually use XNNNN for our own timers. XNNNN transforms to negative timer numbers (to differentiate them from spec related TNNNN positive ones). It could be this timer was a spec-defined timer (and thus should be TNNNN and postive), but since it's not indicated in previous code it's difficult to say which one it is. If someone at some point finds the related timer, we can rename it.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15425/2/src/tbf.cpp">File src/tbf.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/15425/2/src/tbf.cpp@664">Patch Set #2, Line 664:</a> <code style="font-family:monospace,monospace">enum tbf_timers t,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Do you think we still need this parameter? Most of the time I see: T_START(tbf, TXXX, XXX, ... […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It is still needed unless we change implementation of how osmo_timer objects are stored, which for sure I'm not going to do now in the same patch.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-pcu/+/15425">change 15425</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-pcu/+/15425"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-pcu </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I5cfb9ef01706124be262d4536617b9edb4601dd5 </div>
<div style="display:none"> Gerrit-Change-Number: 15425 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 09 Sep 2019 09:23:32 +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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>