<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/9217">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/9217/1/src/libbsc/bsc_vty.c">File src/libbsc/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/9217/1/src/libbsc/bsc_vty.c@4143">Patch Set #1, Line 4143:</a> <code style="font-family:monospace,monospace">void openbsc_vty_print_statistics(struct vty *vty, struct gsm_network *net)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">We can merge this patch first, but I think it would be actually nice to have this as part of libosmocore.  It should be the same for all applications, as it's truly generic.  libosmovty depends on libosmocore, and rate_ctr are in libosmcoore, i.e. anyone using libosmovty should automatically get the "print statistics" installed.  Maybe we should call it "print rate-counters" instead, as this will not clash with any existing VTY commands code like this installs inside the applications, so we can handle this in a graceful way (old programs with new libosmocore, ...)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/9217">change 9217</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/9217"/><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: I58ae04e1960774d760e3ebb54a4f307c9f753655 </div>
<div style="display:none"> Gerrit-Change-Number: 9217 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 17 May 2018 20:11:06 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>