<p>neels <strong>uploaded patch set #10</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16459">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">2/2: wrap ipa_name in osmo_cni_peer_id with type enum and union<br><br>To be prepared for the future in public API, wrap the new osmo_ipa_name struct<br>in an enum-type and union called osmo_cni_peer.<br><br>During code review it was requested to insert an ability to handle different<br>kinds of peer id, in order to be able to add a Global Title in the future.<br><br>Use the generic osmo_cni_peer only in the publicly visible API. For osmo-hlr<br>internal code, I intend to postpone implementing this into the future, when a<br>different peer identification actually gets introduced.<br><br>This way we don't need to implement it now in all osmo-hlr code paths (save<br>time now), but still make all external API users aware that this type may be<br>extended in the future.<br><br>Change-Id: Ide9dcdca283ab989240cfc6e53e9211862a199c5<br>---<br>M include/Makefile.am<br>A include/osmocom/gsupclient/cni_peer_id.h<br>M include/osmocom/gsupclient/gsup_req.h<br>D include/osmocom/gsupclient/ipa_name.h<br>M include/osmocom/hlr/db.h<br>M include/osmocom/hlr/gsup_server.h<br>M src/Makefile.am<br>M src/db_hlr.c<br>M src/gsup_server.c<br>M src/gsupclient/Makefile.am<br>A src/gsupclient/cni_peer_id.c<br>M src/gsupclient/gsup_req.c<br>D src/gsupclient/ipa_name.c<br>M src/hlr.c<br>M src/hlr_ussd.c<br>M src/lu_fsm.c<br>M tests/db/Makefile.am<br>M tests/db/db_test.c<br>M tests/gsup_server/Makefile.am<br>19 files changed, 346 insertions(+), 181 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-hlr refs/changes/59/16459/10</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16459">change 16459</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-hlr/+/16459"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-hlr </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ide9dcdca283ab989240cfc6e53e9211862a199c5 </div>
<div style="display:none"> Gerrit-Change-Number: 16459 </div>
<div style="display:none"> Gerrit-PatchSet: 10 </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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </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-MessageType: newpatchset </div>