Change in libosmo-netif[master]: prim: Add internal CTL SAPI to negotiate SAP versions

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

pespin gerrit-no-reply at lists.osmocom.org
Thu Dec 9 12:56:32 UTC 2021


pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/26435 )

Change subject: prim: Add internal CTL SAPI to negotiate SAP versions
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.osmocom.org/c/libosmo-netif/+/26435/4/src/prim.c 
File src/prim.c:

https://gerrit.osmocom.org/c/libosmo-netif/+/26435/4/src/prim.c@98 
PS4, Line 98: OSMO_PRIM_CTL_SAPI
> this could be a public #define that is visible to the applications?
I don't think it's really needed so far since anyway this is all handled internally by the module, so totally transparent to the app (the protocol side at least, not the callback). In any case, we can always later on move it to a header without any problem.



-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/26435
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I0c2d92cfdb5433e3caab51d712fd947d51eeef23
Gerrit-Change-Number: 26435
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Comment-Date: Thu, 09 Dec 2021 12:56:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge at osmocom.org>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20211209/2a10f563/attachment.htm>


More information about the gerrit-log mailing list