Attention is currently required from: fixeria.
daniel has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/33495 )
Change subject: osmo_io: Use bitfield for various boolean flags
......................................................................
Patch Set 2:
(5 comments)
File src/core/osmo_io_internal.h:
https://gerrit.osmocom.org/c/libosmocore/+/33495/comment/0f45e21e_cb6c58be
PS1, Line 39: iofd
`(iofd)->flags`, just to be safe
Ack
https://gerrit.osmocom.org/c/libosmocore/+/33495/comment/78a42cab_daf522e8
PS1, Line 40: } while (0)
Agreeing with the linter, do-while wrapper is not
really needed here.
Done
https://gerrit.osmocom.org/c/libosmocore/+/33495/comment/094d0a26_92910ae9
PS1, Line 47: iofd->flags & flag
same here: `(iofd)->flags & (flag)`
Done
https://gerrit.osmocom.org/c/libosmocore/+/33495/comment/8a099afd_04c1b77d
PS1, Line 58: TODO: Move to bitfield
remove this TODO?
Ack
File src/core/osmo_io_poll.c:
https://gerrit.osmocom.org/c/libosmocore/+/33495/comment/c73bcb46_6990a057
PS1, Line 84: FLAG_ISSET
missing `IOFD_` prefix
Done
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/33495
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ic134e4c8d791c34778202fea98a70bc04007a113
Gerrit-Change-Number: 33495
Gerrit-PatchSet: 2
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 27 Jun 2023 14:12:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment