git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/3] compat/poll: prepare for targeting Windows Vista

From
Johannes Schindelin via GitGitGadget <gitgitgadget@gmail.com>
Date
Oct 3, 2018, 19:43 UTC
Message-ID
<1c2dba41189e97c2dfa7d98da3b52589eb9091cc.1538595818.git.gitgitgadget@gmail.com>
In-Reply-To
<pull.44.git.gitgitgadget@gmail.com>
From: Johannes Schindelin <johannes.schindelin@gmx.de>

Windows Vista (and later) actually have a working poll(), but we still cannot use it because it only works on sockets.

So let's detect when we are targeting Windows Vista and undefine those constants, and define `pollfd` so that we can declare our own pollfd struct.

We also need to make sure that we override those constants *after* `winsock2.h` has been `#include`d (otherwise we would not really override those constants).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 compat/poll/poll.c |  6 +++---
 compat/poll/poll.h | 15 +++++++++++++++
 2 files changed, 18 insertions(+), 3 deletions(-)
diff --git a/compat/poll/poll.c b/compat/poll/poll.c
index 7ed3fbbea1..ad5dcde439 100644
--- a/compat/poll/poll.c
+++ b/compat/poll/poll.c
@@ -29,9 +29,6 @@
 
 #include <sys/types.h>
 
-/* Specification.  */
-#include <poll.h>
-
 #include <errno.h>
 #include <limits.h>
 #include <assert.h>
@@ -55,6 +52,9 @@
 # include <unistd.h>
 #endif
 
+/* Specification.  */
+#include "poll.h"
+
 #ifdef HAVE_SYS_IOCTL_H
 # include <sys/ioctl.h>
 #endif
diff --git a/compat/poll/poll.h b/compat/poll/poll.h
index cd1995292a..1e1597360f 100644
--- a/compat/poll/poll.h
+++ b/compat/poll/poll.h
@@ -21,6 +21,21 @@
 #ifndef _GL_POLL_H
 #define _GL_POLL_H
 
+#if defined(_WIN32_WINNT) && _WIN32_WINNT >= 0x600
+/* Vista has its own, socket-only poll() */
+#undef POLLIN
+#undef POLLPRI
+#undef POLLOUT
+#undef POLLERR
+#undef POLLHUP
+#undef POLLNVAL
+#undef POLLRDNORM
+#undef POLLRDBAND
+#undef POLLWRNORM
+#undef POLLWRBAND
+#define pollfd compat_pollfd
+#endif
+
 /* fake a poll(2) environment */
 #define POLLIN      0x0001      /* any readable data available   */
 #define POLLPRI     0x0002      /* OOB/Urgent readable data      */
-- 
gitgitgadget
Previous: Johannes Schindelin via GitGitGadgetNext: Johannes Schindelin via GitGitGadget
Message 2 of 9 in “mingw: require Windows Vista or later (and fix the Windows CI builds)”
  1. 0/3 mingw: require Windows Vista or later (and fix the Windows CI builds)Johannes Schindelin via GitGitGadget, Oct 3, 2018
  2. 1/3 compat/poll: prepare for targeting Windows VistaJohannes Schindelin via GitGitGadget, Oct 3, 2018
  3. 2/3 mingw: set _WIN32_WINNT explicitly for Git for WindowsJohannes Schindelin via GitGitGadget, Oct 3, 2018
  4. 3/3 mingw: bump the minimum Windows version to VistaJohannes Schindelin via GitGitGadget, Oct 3, 2018
  5. Ævar Arnfjörð BjarmasonOct 10, 2018
  6. Junio C HamanoOct 11, 2018
  7. Johannes SchindelinOct 14, 2019
  8. Johannes SchindelinOct 14, 2019
  9. Johannes SchindelinOct 14, 2019

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.