# HG changeset patch
# User Bruno Haible <bruno@clisp.org>
# Date 1306009723 -7200
# Node ID 0e87a6225f1a76975e04ba5196ada8c2b7e6e41d
# Parent  0b9360a3b14bbee10e02d124e38190740ca3a07b
select: Simplify replacement idiom.

* m4/select.m4 (gl_FUNC_SELECT): Set REPLACE_SELECT also on native
Win32 platforms.
* lib/sys_select.in.h (select): Simplify accordingly.
* modules/select (Depends-on): Likewise.

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2011-05-21  Bruno Haible  <bruno@clisp.org>
+
+	select: Simplify replacement idiom.
+	* m4/select.m4 (gl_FUNC_SELECT): Set REPLACE_SELECT also on native
+	Win32 platforms.
+	* lib/sys_select.in.h (select): Simplify accordingly.
+	* modules/select (Depends-on): Likewise.
+
 2011-05-21  Bruno Haible  <bruno@clisp.org>
 
 	mkdir-p: Simplify autoconf macro.
diff --git a/lib/sys_select.in.h b/lib/sys_select.in.h
--- a/lib/sys_select.in.h
+++ b/lib/sys_select.in.h
@@ -89,7 +89,7 @@
 
 
 #if @GNULIB_SELECT@
-# if @HAVE_WINSOCK2_H@ || @REPLACE_SELECT@
+# if @REPLACE_SELECT@
 #  if !(defined __cplusplus && defined GNULIB_NAMESPACE)
 #   undef select
 #   define select rpl_select
diff --git a/m4/select.m4 b/m4/select.m4
--- a/m4/select.m4
+++ b/m4/select.m4
@@ -1,4 +1,4 @@
-# select.m4 serial 3
+# select.m4 serial 4
 dnl Copyright (C) 2009-2011 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -10,6 +10,7 @@
   AC_REQUIRE([AC_CANONICAL_HOST]) dnl for cross-compiles
   AC_REQUIRE([gl_SOCKETS])
   if test "$ac_cv_header_winsock2_h" = yes; then
+    REPLACE_SELECT=1
     AC_LIBOBJ([select])
   else
     dnl On Interix 3.5, select(0, NULL, NULL, NULL, timeout) fails with error
diff --git a/modules/select b/modules/select
--- a/modules/select
+++ b/modules/select
@@ -7,8 +7,8 @@
 
 Depends-on:
 sys_select
-alloca          [test "$ac_cv_header_winsock2_h" = yes || test $REPLACE_SELECT = 1]
-sockets         [test "$ac_cv_header_winsock2_h" = yes || test $REPLACE_SELECT = 1]
+alloca          [test $REPLACE_SELECT = 1]
+sockets         [test $REPLACE_SELECT = 1]
 
 configure.ac:
 gl_FUNC_SELECT