<p style="white-space: pre-wrap; word-wrap: break-word;">This API, like I think I described quite verbosely, breaks the idea of a SAP.  The idea of a SAP is that there's no shared state between the provider and the user of the SAP, and that any communication between provider and user happens by means of asynchronous primitives.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, as I equally explained, we should neither expose nor document (in the commitlog, ...) that osmo_sccp currently simply uses the connection identifier (local significance on the SAP) as SCCP local reference on the wire.  That's a hack, and nobody should ever rely on it, or make that assumption.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also, as we have seen in the related discussion, this is introducing bugs as the SCCP local reference is scoped to the SCCP user, while the "connection id" like the SAP is scoped to the SCCP instance.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/13278">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13278">change 13278</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/13278"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I6e1b4897f4143b97bf90bfd8d0ae5d50bbd2e1dd </div>
<div style="display:none"> Gerrit-Change-Number: 13278 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 15 Mar 2019 17:24:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>