<p><a href="https://gerrit.osmocom.org/13017">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/13017/1/src/libmsc/sgs_vty.c">File src/libmsc/sgs_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/13017/1/src/libmsc/sgs_vty.c@163">Patch Set #1, Line 163:</a> <code style="font-family:monospace,monospace">       if (strcmp(sgs->cfg.local_addr, DEFAULT_SGS_SERVER_IP))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think it makes more sense to drop these conditions and print default values too, as we do in the 'hlr' section. The VTY tests expect default values to be printed, excluding port.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13017">change 13017</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/13017"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If045e42fca0315b0777eb86c44bf934ce58b340b </div>
<div style="display:none"> Gerrit-Change-Number: 13017 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 23 Feb 2019 09:54:23 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>