Attention is currently required from: osmith, Hoernchen, neels, laforge, 
pespin, fixeria.

daniel has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/30934 )

Change subject: Add osmo_io with initial poll backend
......................................................................


Patch Set 8:

(3 comments)

File src/core/osmo_io.c:

https://gerrit.osmocom.org/c/libosmocore/+/30934/comment/3806d858_1cc24bd2 
PS7, Line 5: 2022
> probably also 2023 now. Not super important.
Ack


https://gerrit.osmocom.org/c/libosmocore/+/30934/comment/975d52b3_a82d08b7
PS7, Line 349: iofd(%s)
> I see multiple log lines manually encoding this prefix. […]
Yeah, I also thought about that.


https://gerrit.osmocom.org/c/libosmocore/+/30934/comment/c70fa3fe_9f428703
PS7, Line 432: using backend %s
> as the backend is global, I'm not sure we need to log this every time?
ACK, I guess that's left over from an older version where it wasn't yet global



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I50d73cf550d6ce8154bf827bf47408131cf5b0a0
Gerrit-Change-Number: 30934
Gerrit-PatchSet: 8
Gerrit-Owner: daniel <dwillm...@sysmocom.de>
Gerrit-Reviewer: Hoernchen <ew...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-CC: laforge <lafo...@osmocom.org>
Gerrit-Attention: osmith <osm...@sysmocom.de>
Gerrit-Attention: Hoernchen <ew...@sysmocom.de>
Gerrit-Attention: neels <nhofm...@sysmocom.de>
Gerrit-Attention: laforge <lafo...@osmocom.org>
Gerrit-Attention: pespin <pes...@sysmocom.de>
Gerrit-Attention: fixeria <vyanits...@sysmocom.de>
Gerrit-Comment-Date: Sat, 01 Apr 2023 12:27:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <lafo...@osmocom.org>
Gerrit-MessageType: comment

Reply via email to