<p><a href="https://gerrit.osmocom.org/11654">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11654/3/src/osmo-bsc/bsc_vty.c">File src/osmo-bsc/bsc_vty.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/11654/3/src/osmo-bsc/bsc_vty.c@3893">Patch Set #3, Line 3893:</a> <code style="font-family:monospace,monospace">      "Calculate T3113 dynamically based on channel config and load\n"</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I think it would be good to make the name of this command more specific to t3113, for now.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">"for now" means having to keep backward compatibility with commands later, and I want to avoid that. So indeed the implementation is so far quite specific to T3113, but same VTY cmd can be used later on for more timers if we decide so, without needing to add more commands or deprecating all ones.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11654/3/src/osmo-bsc/bsc_vty.c@3916">Patch Set #3, Line 3916:</a> <code style="font-family:monospace,monospace">   "Calculate T3113 dynamically based on channel config and load\n"</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">with the preceding "no", it's ok to here write "Set given timer to non-dynamic and use the default o […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I didn't completely get what you mean here regarding docs, but anyway that sounds confusing to me when reading because I would read:<br>NO_STR "Set given timer to non-dynamic and use the default or user provided fixed value"</p><p style="white-space: pre-wrap; word-wrap: break-word;">which in my head would mean:<br>"Don't set given timer to non-dynamic and use the default or user provided fixed value"</p><p style="white-space: pre-wrap; word-wrap: break-word;">which by boolean !(!a&&b):<br>"Set given timer to dynamic or don't use default provided fixed value"</p><p style="white-space: pre-wrap; word-wrap: break-word;">And that's basically the opposite of what this command does.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11654/3/src/osmo-bsc/bsc_vty.c@3927">Patch Set #3, Line 3927:</a> <code style="font-family:monospace,monospace">           vty_out(vty, "T%d has no dynamic setting%s", d->T, VTY_NEWLINE);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">technically would be ok to ignore this, because setting a non-dynamic timer to non-dynamic is a no-o […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Sure, but anyway nowadays doesn't make sense, to let's better tell the user probably he's doing something having no effect (so he doesn't expect an effect, or detects he mistyped the timer reference).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11654">change 11654</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/11654"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I4fb2969b690151415038631fb6ad059aa6835c7f </div>
<div style="display:none"> Gerrit-Change-Number: 11654 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@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: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 22 Nov 2018 18:06:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>