Attention is currently required from: arehbein, daniel.
View Change
1 comment:
Patchset:
Patch Set #3:
I realized that some of my earlier comments were only applicable if you reviewed this current patch and the next patch. This intermediate stage actually uses the fd for other things than the sls, and hence I've put an OSMO_ASSERT above. all of this will be removed in the follow-up patch, so we don't need to debate of the assert is too aggressive or we should rather have error handling.
To view, visit change 35073. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I12c66badfb4bdfdfe71f1716de960d353d3548b1
Gerrit-Change-Number: 35073
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: arehbein <arehbein@sysmocom.de>
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Attention: arehbein <arehbein@sysmocom.de>
Gerrit-Attention: daniel <dwillmann@sysmocom.de>
Gerrit-Comment-Date: Sat, 25 Nov 2023 16:32:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment