<p style="white-space: pre-wrap; word-wrap: break-word;">Thanks for all the reviews!</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Is there any 'mgcp_vty_reference.xml'? If yes, please also update it.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'll do that in a follow-up patch.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Giving +2, because of >2x +1 (see https://osmocom.org/projects/cellular-infrastructure/wiki/Gerrit#Voting-Rules-for-merging-a-patch-to-master ).</p><p>Patch set 3:<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/c/osmo-mgw/+/14601">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/14601/1/src/libosmo-mgcp/mgcp_vty.c">File src/libosmo-mgcp/mgcp_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/14601/1/src/libosmo-mgcp/mgcp_vty.c@1392">Patch Set #1, Line 1392:</a> <code style="font-family:monospace,monospace">DEFUN(cfg_mgcp_no_conn_timeout,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">We should agree on one way to do it, because "no e1_line 0 keepalive" is still what libosmo-abis wan […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I would also like to have this consistent, but unfortunately we have both versions already. Here are two examples of <0-...:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> osmo-hlr/src/hlr_vty.c:        "ncss-guard-timeout <0-255>",<br> osmo-msc/src/libmsc/msc_vty.c:      "mncc guard-timeout <0-255>",</pre><p style="white-space: pre-wrap; word-wrap: break-word;">So I'll go with the <0-... version here instead of changing it again.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe it makes sense to propose a standard way of doing it (on the ML) and then deprecating the other one?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/14601">change 14601</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-mgw/+/14601"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I7dee7dafaaf4bb93fd692ea06b52b9e012beac6d </div>
<div style="display:none"> Gerrit-Change-Number: 14601 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Hoernchen <ewild@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: laforge <laforge@gnumonks.org> </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-Comment-Date: Thu, 27 Jun 2019 07:23:39 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>