[Asterisk-code-review] codec dahdi: Fix poll.h include. (asterisk[master])

Joshua Colp asteriskteam at digium.com
Mon Nov 28 10:24:24 CST 2016


Joshua Colp has submitted this change and it was merged. ( https://gerrit.asterisk.org/4466 )

Change subject: codec_dahdi: Fix poll.h include.
......................................................................


codec_dahdi: Fix poll.h include.

POSIX defines poll.h. sys/poll.h should not be used as it is c-library
internal header which may or may not exist. Notably in musl including
sys/poll.h generates warning of being incorrect.

Change-Id: Ib318c1c7142a737bcf3caa4d8d72560bebe39252
---
M codecs/codec_dahdi.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  George Joseph: Looks good to me, approved
  Anonymous Coward #1000019: Verified
  Joshua Colp: Looks good to me, but someone else must approve



diff --git a/codecs/codec_dahdi.c b/codecs/codec_dahdi.c
index 2d08d36..efb0168 100644
--- a/codecs/codec_dahdi.c
+++ b/codecs/codec_dahdi.c
@@ -34,11 +34,11 @@
 #include "asterisk.h"
 #include <stdbool.h>
 
+#include <poll.h>
 #include <fcntl.h>
 #include <netinet/in.h>
 #include <sys/ioctl.h>
 #include <sys/mman.h>
-#include <sys/poll.h>
 #include <dahdi/user.h>
 
 #include "asterisk/lock.h"

-- 
To view, visit https://gerrit.asterisk.org/4466
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib318c1c7142a737bcf3caa4d8d72560bebe39252
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Timo Teräs <timo.teras at iki.fi>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list