Attention is currently required from: laforge, fixeria, daniel.

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

Change subject: stream: Split cli/srv specific code out of stream.c
......................................................................


Patch Set 3:

(1 comment)

Patchset:

PS3:
> are we sure this re-factoring doesn't clash with ongoing osmo_io work? We 
> should always avoid to do  […]
Most of the io work was already merged, at least the initial big commits, so I 
expect minimal crash now.
In any case, I'm also doing a lot of ongoing work now in order to improve 
libosmo-sccp Primary Address on top of this commit. I think I already provided 
a big amount of time (lots of weeks, even months?) to get whatever work there 
merged.

Furthermore, this split should also make it easier to submit new patches 
regarding io in the future.



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

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I22af01bba2040eb320ba48fd1b46c090c98be159
Gerrit-Change-Number: 34057
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Attention: daniel <[email protected]>
Gerrit-Comment-Date: Fri, 04 Aug 2023 15:48:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment

Reply via email to