<p>osmith <strong>uploaded patch set #8</strong> to the change originally created by Neels Hofmeyr.</p><p><a href="https://gerrit.osmocom.org/13123">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">add osmo_sockaddr_str API<br><br>For handling RTP IP addresses and ports, osmo-mgw, osmo-bsc and osmo-msc<br>so far have their own separate shims and code duplication around<br>inet_ntoa(), htons(), sockaddr conversions etc. Unify and standardize<br>with this common API.<br><br>In the MGW endpoint FSM that was introduced in osmo-bsc and which I<br>would like to re-use for osmo-msc (upcoming patch moving that to<br>osmo-mgw), it has turned out that using char* IP address and uint16_t<br>port number types are a convenient common denominator for logging,<br>MGCP message composition and GSM48. Ongoing osmo-msc work also uses this<br>for MNCC.<br><br>This is of course potentially useful for any other IP+port combinations<br>besides RTP stream handling.<br><br>Needless to say that most current implementations will probably stay<br>with their current own conversion code for a long time; for current<br>osmo-{bsc,msc,mgw} work (MGW endpoint FSM) though, I would like to move<br>to this API here.<br><br>Change-Id: Id617265337f09dfb6ddfe111ef5e578cd3dc9f63<br>---<br>M configure.ac<br>M include/Makefile.am<br>A include/osmocom/core/sockaddr_str.h<br>M src/Makefile.am<br>A src/sockaddr_str.c<br>M tests/Makefile.am<br>A tests/sockaddr_str/sockaddr_str_test.c<br>A tests/sockaddr_str/sockaddr_str_test.ok<br>M tests/testsuite.at<br>9 files changed, 1,008 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/23/13123/8</pre><p>To view, visit <a href="https://gerrit.osmocom.org/13123">change 13123</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/13123"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Id617265337f09dfb6ddfe111ef5e578cd3dc9f63 </div>
<div style="display:none"> Gerrit-Change-Number: 13123 </div>
<div style="display:none"> Gerrit-PatchSet: 8 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-CC: osmith <osmith@sysmocom.de> </div>