[zeromq-dev] [PATCH] FD_SETSIZE increased for win32

[Mikael Helbo Kjær]

> Additionally, I would suggest using a following simpler patch:
> 
> index 78d3afd..663d5d7 100644
> --- a/src/select.cpp
> +++ b/src/select.cpp
> @@ -23,6 +23,7 @@
>   #include <algorithm>
> 
>   #ifdef ZMQ_HAVE_WINDOWS
> +#define FD_SETSIZE 1024
>   #include "winsock2.h"
>   #elif defined ZMQ_HAVE_HPUX
>   #include <sys/param.h>
> 

Please do it like this instead

+#ifndef FD_SETSIZE
+#  define FD_SETSIZE 1024
+#endif

Some of us are already setting FD_SETSIZE as a compile time value so that would give us warnings and other problems (if we have a larger or smaller FD_SETSIZE due to other code or the like).

Regards,
Mikael