<p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25335">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25335/4//COMMIT_MSG">Commit Message:</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-mgw/+/25335/4//COMMIT_MSG@18">Patch Set #4, Line 18:</a> <code style="font-family:monospace,monospace">  and mgcp_client_pool.c</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">(Two logical changes in one commit, reviewing would have been easier if they were separate. But probably not worth changing now, so fine with me to keep it in one here.)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25335/4/src/libosmo-mgcp-client/mgcp_client.c">File src/libosmo-mgcp-client/mgcp_client.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-mgw/+/25335/4/src/libosmo-mgcp-client/mgcp_client.c@1513">Patch Set #4, Line 1513:</a> <code style="font-family:monospace,monospace">mame</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">name</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25335/4/src/libosmo-mgcp-client/mgcp_client_pool.c">File src/libosmo-mgcp-client/mgcp_client_pool.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-mgw/+/25335/4/src/libosmo-mgcp-client/mgcp_client_pool.c@54">Patch Set #4, Line 54:</a> <code style="font-family:monospace,monospace">        snprintf(name, sizeof(name), "%d:%s", pool_member->nr, description);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">the code to obtain description seems to be duplicated in mgcp_client_name(). So just use that function here?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/25335">change 25335</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/+/25335"/><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: I53ff5445c8e5faffa4ef908ffb1fdb1f47ea2904 </div>
<div style="display:none"> Gerrit-Change-Number: 25335 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@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-CC: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 13 Sep 2021 11:14:41 +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>