<p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16932">View Change</a></p><p>4 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16932/1/src/libmsc/msc_vty.c">File src/libmsc/msc_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-msc/+/16932/1/src/libmsc/msc_vty.c@679">Patch Set #1, Line 679:</a> <code style="font-family:monospace,monospace">       return osmo_tdef_vty_set_cmd(vty, tdefs, argv + 1);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">argv[1]?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16932/1/src/libmsc/msc_vty.c@710">Patch Set #1, Line 710:</a> <code style="font-family:monospace,monospace">  return osmo_tdef_vty_set_cmd(vty, tdefs, argv + 1);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">argv[1] ?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16932/1/src/libmsc/msc_vty.c@844">Patch Set #1, Line 844:</a> <code style="font-family:monospace,monospace">         osmo_tdef_vty_out_all(vty, list, "%*c", offset, ' '); \</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ugh, what does "%*c" mean?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-msc/+/16932/1/src/libmsc/msc_vty.c@848">Patch Set #1, Line 848:</a> <code style="font-family:monospace,monospace">      "show timer [all]",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">AFAIu all here is not needed at all so it can be dropped? Or otherwise leave it as mandatory, but having it as optional is confusing, as the user thinks the behavior is different.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/16932">change 16932</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-msc/+/16932"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I6024c104b6101666c8aa1108a043910eb75db9a5 </div>
<div style="display:none"> Gerrit-Change-Number: 16932 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 20 Jan 2020 10:30:06 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>