pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/41928?usp=email )


Change subject: osmo_io.h: add forward declaration of struct osmo_sockaddr
......................................................................

osmo_io.h: add forward declaration of struct osmo_sockaddr

During amr-none-eabi build (EMBEDDED defined), including
<osmocom/core/socket.h> includes a mostly empty file, since that file is
guarded by "#if (!EMBEDDED)" because sys/socket.h is not available
there. As a result, if somebody includes osmo_io.h when EMBEDDED is set,
there will be no declaration of struct osmo_sockaddr used in declared
APIs. Add a forward declaration to fix it.

Change-Id: Id7db4b081752a1034b95a92b4850484a26ec11c6
---
M include/osmocom/core/osmo_io.h
1 file changed, 4 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/28/41928/1

diff --git a/include/osmocom/core/osmo_io.h b/include/osmocom/core/osmo_io.h
index 525c23c..b5e7ea8 100644
--- a/include/osmocom/core/osmo_io.h
+++ b/include/osmocom/core/osmo_io.h
@@ -14,6 +14,10 @@
  * We only really need a forward declaration here: */
 struct msghdr;

+/* struct osmo_sockaddr is defined in osmocom/core/socket.h, which in turn 
relies on sys/socket.h to define it,
+ * which may not be available. We only really need a forward declaration here: 
*/
+struct osmo_sockaddr;
+
 /*! \defgroup osmo_io Osmocom I/O interface
  *  @{
  *

--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/41928?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Id7db4b081752a1034b95a92b4850484a26ec11c6
Gerrit-Change-Number: 41928
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>

Reply via email to