<p style="white-space: pre-wrap; word-wrap: break-word;">To summarize:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">- keith had no time to look at this, says to go ahead and deal with the merged situation later.<br>- pespin would prefer an opaque data structure for peer id, for ABI compat reasons.<br>- I'm reluctant to add opaque data because that requires dynamic allocation.<br>  Concerning future ABI compatibility, I agree that it would be a nice thing but is not a hard requirement. IMHO it isn't worth adding dynamic allocation (and rewriting many patches on this branch) for it.<br>- laforge indicates slight preference of the patch staying as it is now.<br>- I would actually prefer a plain char string (the current real world usage).<br>  My gut feel is that this will remain API bloat forever, but it gives the benefit of the doubt to being able to expand the API compatibly in the future.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">Seems like everyone is pointing in a slightly different direction.<br>But I think we need to come to a conclusion.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In the last patch set, I only renamed osmo_gsup_peer_id to osmo_cni_peer_id, added some API doc and tweaked some commit log messages, added a const.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So, essentially, these patches are still the same that they have been for months</p><p style="white-space: pre-wrap; word-wrap: break-word;">I've probed the patches for desirable changes for a while and seem to hit reasons to not change anything in every direction. So we either find agreement on how to change these patches, or we merge them as they are. I think there is agreement that we don't want to keep this on a branch for much longer, so I'd ask reviewers to converge on a verdict, ideally one that says CR+2...</p><p><a href="https://gerrit.osmocom.org/c/osmo-hlr/+/16459">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><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: 11 </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-Comment-Date: Thu, 30 Apr 2020 17:31:08 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>