<p style="white-space: pre-wrap; word-wrap: break-word;">I really do think cosnistency in the VTY is important and I often blame myself for not having caught something at code review which is now quite inconsistenc in the VTY.  Let's make sure we get this right before submission.</p><p>Patch set 12:<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-bsc/+/24374">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-bsc/+/24374/10/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/osmo-bsc/+/24374/10/src/osmo-bsc/bsc_vty.c@2174">Patch Set #10, Line 2174:</a> <code style="font-family:monospace,monospace">modify lchan</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">multiple words: then what about "handover1 power budget interval <1-99>"? ;) […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The "handover1 power budget interval <1-99>" and the like are annoying early mistakes from a long time ago when at least I did not yet understand fully the nature of VTY (or its father-in-kind of Cisco command line interfaces).  These days, whenever I see something like that, I find it almost embarrassing that we let this past code review in the first place.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I also find the ovrall lack of consistency a serious problem.  It's not about everyone putting their own style in somewhere.  The user expects consistency.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The more I thik about this, the more it should be </p><p style="white-space: pre-wrap; word-wrap: break-word;">"bts <0-255> trx <0-255> timeslot <0-7> sub-slot <0-7> vamos modify-lchan" or the like.  The object is specified first and then the action. This way you first specify the TRX/TS/sub-slot/... in a unique way (you iterate the tree of the data structure) and then you get a choice of actions you can do with that specific object.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't really care whether the trx / timeslot verbs are explicitly spelled or not.  However, as that is what we have in place for now, new commands should follow it.  And in the worst case we could always add a second </p><p style="white-space: pre-wrap; word-wrap: break-word;">"lchan <0-255> <0-255> <0-7> <0-1>" entry path which then accepts the same action commands below it.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24374">change 24374</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-bsc/+/24374"/><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-Change-Id: Ibf53f4797d7491b17a33946fd7d920f038362b4c </div>
<div style="display:none"> Gerrit-Change-Number: 24374 </div>
<div style="display:none"> Gerrit-PatchSet: 12 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 01 Jun 2021 13:53:44 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>