<p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24145">View Change</a></p><p>2 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/+/24145/1/src/libosmo-mgcp-client/mgcp_client_endpoint_fsm.c">File src/libosmo-mgcp-client/mgcp_client_endpoint_fsm.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/+/24145/1/src/libosmo-mgcp-client/mgcp_client_endpoint_fsm.c@511">Patch Set #1, Line 511:</a> <code style="font-family:monospace,monospace">/*! Return the RTP port information that MGW is receiving on for this connection, as returned by the last CRCX-OK /</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">"the RTP port where MGW is receiving on" is IMHO the clearest description... […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes please, "local" and "remote" is a usual terminology which helps understanding quickly.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24145/1/src/libosmo-mgcp-client/mgcp_client_endpoint_fsm.c@523">Patch Set #1, Line 523:</a> <code style="font-family:monospace,monospace">/*! Return the RTP port information that the MGW is sending to for this connection, as sent to the MGW by the last</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">hmm "MGW's remote RTP port" is again a bit weird, because it is the port of the MGW's peer, not belo […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">"MGW's remote RTP port".</p><p style="white-space: pre-wrap; word-wrap: break-word;">The problem seems to be that in libosmo-mgcp-client, the "local" port would be the one bound in the client, and the remote would be the one listening on the MGW. So it may actually may sense to change the API name here...</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24145">change 24145</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/+/24145"/><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: Ibb488925827d9dc0ccb1f8d6d84728745d086793 </div>
<div style="display:none"> Gerrit-Change-Number: 24145 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 06 May 2021 12:59:14 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>