Attention is currently required from: neels, fixeria.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmo-netif/+/29028 )
Change subject: osmux.h: Add missing msgb.h header
......................................................................
Patch Set 1:
(1 comment)
File include/osmocom/netif/osmux.h:
https://gerrit.osmocom.org/c/libosmo-netif/+/29028/comment/a6b4afa6_f4043158
PS1, Line 6: #include <osmocom/core/msgb.h>
adding a forward declaration of 'struct msgb'
would have been enough.
I don't like adding forward declarations of structs
coming from other libraries. I don't think it's the right thing to do.
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-netif/+/29028
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I2002ed2352ef4db2fe0717389f1f6e24a88a973f
Gerrit-Change-Number: 29028
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 11 Aug 2022 13:59:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment