From 4f1d4044f8409ff29eeb7f97324daba496e40714 Mon Sep 17 00:00:00 2001 From: Maarten Bosmans Date: Tue, 4 Jan 2011 17:03:13 +0100 Subject: Use instead of The check whether POSIX socket.h or WIN32 winsock2.h must be included can be made centrally. The downside is that some functionality of e.g. arpa/inet.h is also implemented in winsock.h, so that some files that don't use socket functions, but do use inet.h functions, must also include pulsecore/socket.h. (as well as arpa/inet.h) --- src/Makefile.am | 2 +- src/daemon/main.c | 2 +- src/modules/module-esound-sink.c | 2 +- src/modules/module-protocol-stub.c | 5 +---- src/pulse/context.c | 6 +----- src/pulse/mainloop.c | 2 +- src/pulse/timeval.c | 1 - src/pulse/util.c | 16 ++++++---------- src/pulsecore/core-util.c | 3 +-- src/pulsecore/core-util.h | 2 +- src/pulsecore/creds.h | 4 +--- src/pulsecore/inet_ntop.c | 6 +----- src/pulsecore/inet_ntop.h | 6 +----- src/pulsecore/inet_pton.c | 6 +----- src/pulsecore/inet_pton.h | 6 +----- src/pulsecore/iochannel.c | 7 ++----- src/pulsecore/ioline.c | 2 +- src/pulsecore/ipacl.c | 7 ++----- src/pulsecore/parseaddr.c | 4 +++- src/pulsecore/pipe.c | 6 +----- src/pulsecore/poll.c | 11 +++++------ src/pulsecore/pstream.c | 6 +----- src/pulsecore/rtpoll.c | 1 - src/pulsecore/socket-client.c | 7 ++----- src/pulsecore/socket-server.c | 11 +++-------- src/pulsecore/socket-util.c | 8 ++------ src/pulsecore/socket-util.h | 2 +- src/pulsecore/socket.h | 31 +++++++++++++++++++++++++++++++ src/pulsecore/tagstruct.c | 2 +- src/pulsecore/winsock.h | 26 -------------------------- src/tests/ipacl-test.c | 8 ++------ 31 files changed, 76 insertions(+), 132 deletions(-) create mode 100644 src/pulsecore/socket.h delete mode 100644 src/pulsecore/winsock.h (limited to 'src') diff --git a/src/Makefile.am b/src/Makefile.am index 0687e409..baba9c5e 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -617,7 +617,7 @@ libpulsecommon_@PA_MAJORMINOR@_la_SOURCES = \ pulsecore/tokenizer.c pulsecore/tokenizer.h \ pulsecore/usergroup.c pulsecore/usergroup.h \ pulsecore/sndfile-util.c pulsecore/sndfile-util.h \ - pulsecore/winsock.h + pulsecore/socket.h libpulsecommon_@PA_MAJORMINOR@_la_CFLAGS = $(AM_CFLAGS) $(LIBSAMPLERATE_CFLAGS) $(LIBSNDFILE_CFLAGS) libpulsecommon_@PA_MAJORMINOR@_la_LDFLAGS = $(AM_LDFLAGS) -avoid-version diff --git a/src/daemon/main.c b/src/daemon/main.c index 0db794bc..de809b1b 100644 --- a/src/daemon/main.c +++ b/src/daemon/main.c @@ -70,7 +70,7 @@ #include #include -#include +#include #include #include #include diff --git a/src/modules/module-esound-sink.c b/src/modules/module-esound-sink.c index 0d04fff7..9eba8ac3 100644 --- a/src/modules/module-esound-sink.c +++ b/src/modules/module-esound-sink.c @@ -31,7 +31,6 @@ #include #include #include -#include #include #include #include @@ -44,6 +43,7 @@ #include #include +#include #include #include #include diff --git a/src/modules/module-protocol-stub.c b/src/modules/module-protocol-stub.c index e21186bb..7ba54054 100644 --- a/src/modules/module-protocol-stub.c +++ b/src/modules/module-protocol-stub.c @@ -30,9 +30,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_ARPA_INET_H #include #endif @@ -42,9 +39,9 @@ #include -#include #include #include +#include #include #include #include diff --git a/src/pulse/context.c b/src/pulse/context.c index 7a539b4d..ca816507 100644 --- a/src/pulse/context.c +++ b/src/pulse/context.c @@ -39,9 +39,6 @@ #include #endif -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_SYS_UN_H #include #endif @@ -57,9 +54,7 @@ #include #include -#include #include - #include #include #include @@ -69,6 +64,7 @@ #include #include #include +#include #include #include #include diff --git a/src/pulse/mainloop.c b/src/pulse/mainloop.c index 8f743ec7..8c261381 100644 --- a/src/pulse/mainloop.c +++ b/src/pulse/mainloop.c @@ -47,7 +47,7 @@ #include #include #include -#include +#include #include #include "mainloop.h" diff --git a/src/pulse/timeval.c b/src/pulse/timeval.c index cde4417c..c4a08419 100644 --- a/src/pulse/timeval.c +++ b/src/pulse/timeval.c @@ -31,7 +31,6 @@ #include #endif -#include #include #include diff --git a/src/pulse/util.c b/src/pulse/util.c index ca766dab..3206e94c 100644 --- a/src/pulse/util.c +++ b/src/pulse/util.c @@ -37,10 +37,6 @@ #include #endif -#ifdef HAVE_SYS_SOCKET_H -#include -#endif - #ifdef HAVE_NETDB_H #include #endif @@ -56,7 +52,7 @@ #include #include -#include +#include #include #include #include @@ -195,11 +191,11 @@ char *pa_get_binary_name(char *s, size_t l) { { char *rp; - if ((rp = pa_readlink("/proc/curproc/file"))) { - pa_strlcpy(s, pa_path_get_filename(rp), l); - pa_xfree(rp); - return s; - } + if ((rp = pa_readlink("/proc/curproc/file"))) { + pa_strlcpy(s, pa_path_get_filename(rp), l); + pa_xfree(rp); + return s; + } } #endif diff --git a/src/pulsecore/core-util.c b/src/pulsecore/core-util.c index ec0b338d..3ce5edbf 100644 --- a/src/pulsecore/core-util.c +++ b/src/pulsecore/core-util.c @@ -43,7 +43,6 @@ #include #include #include -#include #ifdef HAVE_STRTOF_L #include @@ -110,7 +109,7 @@ #include #include -#include +#include #include #include #include diff --git a/src/pulsecore/core-util.h b/src/pulsecore/core-util.h index 0d63cfcb..8619330d 100644 --- a/src/pulsecore/core-util.h +++ b/src/pulsecore/core-util.h @@ -28,7 +28,6 @@ #include #include #include -#include #ifdef HAVE_SYS_RESOURCE_H #include @@ -36,6 +35,7 @@ #include #include +#include #ifndef PACKAGE #error "Please include config.h before including this file!" diff --git a/src/pulsecore/creds.h b/src/pulsecore/creds.h index c15c469b..9e6fb124 100644 --- a/src/pulsecore/creds.h +++ b/src/pulsecore/creds.h @@ -28,9 +28,7 @@ #error "Please include config.h before including this file!" #endif -#ifdef HAVE_SYS_SOCKET_H -#include -#endif +#include #ifdef HAVE_SYS_UN_H #include diff --git a/src/pulsecore/inet_ntop.c b/src/pulsecore/inet_ntop.c index 012a1a05..e283a670 100644 --- a/src/pulsecore/inet_ntop.c +++ b/src/pulsecore/inet_ntop.c @@ -28,11 +28,7 @@ #ifndef HAVE_INET_NTOP -#ifdef HAVE_SYS_SOCKET_H -#include -#endif - -#include "winsock.h" +#include #include "inet_ntop.h" diff --git a/src/pulsecore/inet_ntop.h b/src/pulsecore/inet_ntop.h index 7fb67b44..7e1b2037 100644 --- a/src/pulsecore/inet_ntop.h +++ b/src/pulsecore/inet_ntop.h @@ -1,11 +1,7 @@ #ifndef fooinet_ntophfoo #define fooinet_ntophfoo -#ifdef HAVE_SYS_SOCKET_H -#include -#endif - -#include "winsock.h" +#include const char *inet_ntop(int af, const void *src, char *dst, socklen_t cnt); diff --git a/src/pulsecore/inet_pton.c b/src/pulsecore/inet_pton.c index abdfa467..198d1a0d 100644 --- a/src/pulsecore/inet_pton.c +++ b/src/pulsecore/inet_pton.c @@ -28,11 +28,7 @@ #ifndef HAVE_INET_PTON -#ifdef HAVE_SYS_SOCKET_H -#include -#endif - -#include "winsock.h" +#include #include "inet_pton.h" diff --git a/src/pulsecore/inet_pton.h b/src/pulsecore/inet_pton.h index 111b4a07..9d9c476a 100644 --- a/src/pulsecore/inet_pton.h +++ b/src/pulsecore/inet_pton.h @@ -1,11 +1,7 @@ #ifndef fooinet_ptonhfoo #define fooinet_ptonhfoo -#ifdef HAVE_SYS_SOCKET_H -#include -#endif - -#include "winsock.h" +#include int inet_pton(int af, const char *src, void *dst); diff --git a/src/pulsecore/iochannel.c b/src/pulsecore/iochannel.c index f85c9896..f89b0670 100644 --- a/src/pulsecore/iochannel.c +++ b/src/pulsecore/iochannel.c @@ -28,19 +28,16 @@ #include #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif + #ifdef HAVE_SYS_UN_H #include #endif -#include "winsock.h" - #include #include #include +#include #include #include #include diff --git a/src/pulsecore/ioline.c b/src/pulsecore/ioline.c index 7afdb08c..963f7d22 100644 --- a/src/pulsecore/ioline.c +++ b/src/pulsecore/ioline.c @@ -30,7 +30,7 @@ #include -#include +#include #include #include #include diff --git a/src/pulsecore/ipacl.c b/src/pulsecore/ipacl.c index 312e0403..0cbe34af 100644 --- a/src/pulsecore/ipacl.c +++ b/src/pulsecore/ipacl.c @@ -28,9 +28,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_NETINET_IN_H #include #endif @@ -50,10 +47,10 @@ #include #include #include -#include +#include #ifndef HAVE_INET_PTON -#include "inet_pton.h" +#include #endif #include "ipacl.h" diff --git a/src/pulsecore/parseaddr.c b/src/pulsecore/parseaddr.c index 44cd9a05..7f6cd903 100644 --- a/src/pulsecore/parseaddr.c +++ b/src/pulsecore/parseaddr.c @@ -25,8 +25,10 @@ #include #include + +#ifdef HAVE_ARPA_INET_H #include -#include +#endif #include #include diff --git a/src/pulsecore/pipe.c b/src/pulsecore/pipe.c index c21e4b1f..db8e16f8 100644 --- a/src/pulsecore/pipe.c +++ b/src/pulsecore/pipe.c @@ -29,11 +29,7 @@ #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif - -#include "winsock.h" +#include #include "pipe.h" diff --git a/src/pulsecore/poll.c b/src/pulsecore/poll.c index b98fb169..b993c478 100644 --- a/src/pulsecore/poll.c +++ b/src/pulsecore/poll.c @@ -45,8 +45,7 @@ #include #endif -#include "winsock.h" - +#include #include #include @@ -187,11 +186,11 @@ int pa_poll (struct pollfd *fds, unsigned long int nfds, int timeout) { * connected socket, a server socket, or something else using a * 0-byte recv, and use ioctl(2) to detect POLLHUP. */ r = recv(f->fd, NULL, 0, MSG_PEEK); - if (r == 0 || (r < 0 && errno == ENOTSOCK)) - ioctl(f->fd, FIONREAD, &r); + if (r == 0 || (r < 0 && errno == ENOTSOCK)) + ioctl(f->fd, FIONREAD, &r); - if (r == 0) - f->revents |= POLLHUP; + if (r == 0) + f->revents |= POLLHUP; #else /* !OS_IS_DARWIN */ if (recv (f->fd, data, 64, MSG_PEEK) == -1) { if (errno == ESHUTDOWN || errno == ECONNRESET || diff --git a/src/pulsecore/pstream.c b/src/pulsecore/pstream.c index 3e0bfa3b..369e22ca 100644 --- a/src/pulsecore/pstream.c +++ b/src/pulsecore/pstream.c @@ -28,9 +28,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_SYS_UN_H #include #endif @@ -38,10 +35,9 @@ #include #endif - #include -#include +#include #include #include #include diff --git a/src/pulsecore/rtpoll.c b/src/pulsecore/rtpoll.c index 2ef78736..98d7d625 100644 --- a/src/pulsecore/rtpoll.c +++ b/src/pulsecore/rtpoll.c @@ -40,7 +40,6 @@ #include #include #include -#include #include #include diff --git a/src/pulsecore/socket-client.c b/src/pulsecore/socket-client.c index 7c449bee..31acfd2d 100644 --- a/src/pulsecore/socket-client.c +++ b/src/pulsecore/socket-client.c @@ -32,9 +32,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_SYS_UN_H #include #endif @@ -56,9 +53,9 @@ #include #include -#include -#include +#include #include +#include #include #include #include diff --git a/src/pulsecore/socket-server.c b/src/pulsecore/socket-server.c index 3b7c6973..e45ded04 100644 --- a/src/pulsecore/socket-server.c +++ b/src/pulsecore/socket-server.c @@ -32,9 +32,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_SYS_UN_H #include #ifndef SUN_LEN @@ -54,18 +51,16 @@ #endif #ifndef HAVE_INET_NTOP -#include "inet_ntop.h" +#include #endif - #ifndef HAVE_INET_PTON -#include "inet_pton.h" +#include #endif -#include "winsock.h" - #include #include +#include #include #include #include diff --git a/src/pulsecore/socket-util.c b/src/pulsecore/socket-util.c index 2cc9882a..0b16e0f9 100644 --- a/src/pulsecore/socket-util.c +++ b/src/pulsecore/socket-util.c @@ -36,9 +36,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_SYS_UN_H #include #endif @@ -62,17 +59,16 @@ #endif #ifndef HAVE_INET_NTOP -#include "inet_ntop.h" +#include #endif -#include "winsock.h" - #include #include #include #include #include +#include #include "socket-util.h" diff --git a/src/pulsecore/socket-util.h b/src/pulsecore/socket-util.h index f6d16376..154afd4a 100644 --- a/src/pulsecore/socket-util.h +++ b/src/pulsecore/socket-util.h @@ -24,8 +24,8 @@ ***/ #include -#include +#include #include void pa_socket_peer_to_string(int fd, char *c, size_t l); diff --git a/src/pulsecore/socket.h b/src/pulsecore/socket.h new file mode 100644 index 00000000..0d38bee6 --- /dev/null +++ b/src/pulsecore/socket.h @@ -0,0 +1,31 @@ +#ifndef foopulsecoresockethfoo +#define foopulsecoresockethfoo + +#ifdef HAVE_SYS_SOCKET_H +#include +#endif + +#ifdef HAVE_WINSOCK2_H +#include + +#define ESHUTDOWN WSAESHUTDOWN +#define ECONNRESET WSAECONNRESET +#define ECONNABORTED WSAECONNABORTED +#define ENETRESET WSAENETRESET +#define EINPROGRESS WSAEINPROGRESS +#define EAFNOSUPPORT WSAEAFNOSUPPORT +#define ETIMEDOUT WSAETIMEDOUT +#define ECONNREFUSED WSAECONNREFUSED +#define EHOSTUNREACH WSAEHOSTUNREACH +#define EWOULDBLOCK WSAEWOULDBLOCK +#define EADDRINUSE WSAEADDRINUSE + +typedef long suseconds_t; + +#endif + +#ifdef HAVE_WS2TCPIP_H +#include +#endif + +#endif diff --git a/src/pulsecore/tagstruct.c b/src/pulsecore/tagstruct.c index 330b7596..804b9f90 100644 --- a/src/pulsecore/tagstruct.c +++ b/src/pulsecore/tagstruct.c @@ -35,7 +35,7 @@ #include -#include +#include #include #include "tagstruct.h" diff --git a/src/pulsecore/winsock.h b/src/pulsecore/winsock.h deleted file mode 100644 index 0352bf4d..00000000 --- a/src/pulsecore/winsock.h +++ /dev/null @@ -1,26 +0,0 @@ -#ifndef foowinsockhfoo -#define foowinsockhfoo - -#ifdef HAVE_WINSOCK2_H -#include - -#define ESHUTDOWN WSAESHUTDOWN -#define ECONNRESET WSAECONNRESET -#define ECONNABORTED WSAECONNABORTED -#define ENETRESET WSAENETRESET -#define EINPROGRESS WSAEINPROGRESS -#define EAFNOSUPPORT WSAEAFNOSUPPORT -#define ETIMEDOUT WSAETIMEDOUT -#define ECONNREFUSED WSAECONNREFUSED -#define EHOSTUNREACH WSAEHOSTUNREACH -#define EWOULDBLOCK WSAEWOULDBLOCK - -typedef long suseconds_t; - -#endif - -#ifdef HAVE_WS2TCPIP_H -#include -#endif - -#endif diff --git a/src/tests/ipacl-test.c b/src/tests/ipacl-test.c index 57b70685..88bd6c6f 100644 --- a/src/tests/ipacl-test.c +++ b/src/tests/ipacl-test.c @@ -8,9 +8,6 @@ #include #include -#ifdef HAVE_SYS_SOCKET_H -#include -#endif #ifdef HAVE_NETINET_IN_H #include #endif @@ -24,9 +21,8 @@ #include #endif -#include "../pulsecore/winsock.h" -#include "../pulsecore/macro.h" - +#include +#include #include int main(int argc, char *argv[]) { -- cgit