Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-11-10 Thread Olivier Fourdan
Hi Jon, - Original Message - > Looks like I applied the patch [1]? > > [1] > http://cgit.freedesktop.org/xorg/proto/x11proto/commit/?id=edc04f87c74abacb57782dc947f81a2810a72537 Oh sorry, I overlooked that commit! Thanks! Olivier ___ xorg-devel

Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-11-10 Thread Jon Turney
On 10/11/2015 15:28, Olivier Fourdan wrote: On 26/08/2015 16:24, Olivier Fourdan wrote: The value of FD_SETSIZE for those targets in xserver's configure.ac:403 probably needs to be increased to match (See the comment on xserver commit ed2c2a68) Yeah, well, I changed that already to 512 in comm

Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-11-10 Thread Olivier Fourdan
Hi all - Original Message - > On 26/08/2015 16:24, Olivier Fourdan wrote: > >> The value of FD_SETSIZE for those targets in xserver's configure.ac:403 > >> probably needs to be increased to match (See the comment on xserver > >> commit ed2c2a68) > > > > Yeah, well, I changed that already t

Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-08-26 Thread Jon TURNEY
On 26/08/2015 16:24, Olivier Fourdan wrote: The value of FD_SETSIZE for those targets in xserver's configure.ac:403 probably needs to be increased to match (See the comment on xserver commit ed2c2a68) Yeah, well, I changed that already to 512 in commit d206c240 for consistency (in doubt, I tri

Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-08-26 Thread Olivier Fourdan
Hi Jon, > The value of FD_SETSIZE for those targets in xserver's configure.ac:403 > probably needs to be increased to match (See the comment on xserver > commit ed2c2a68) Yeah, well, I changed that already to 512 in commit d206c240 for consistency (in doubt, I tried to keep consistent). > (The

Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-08-26 Thread Jon TURNEY
On 26/08/2015 09:13, Olivier Fourdan wrote: Sorry, I forgot to amend my commit to indicate this is obviously for "x11proto". Another point is, I have not tested this change on WIN32/CYGWIN as I don't really use these platforms myself... The value of FD_SETSIZE for those targets in xserver's c

Re: [PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-08-26 Thread Olivier Fourdan
Hi Sorry, I forgot to amend my commit to indicate this is obviously for "x11proto". Another point is, I have not tested this change on WIN32/CYGWIN as I don't really use these platforms myself... Cheers, Olivier - Original Message - > Commit 2c94cdb raised the number of file descriptor

[PATCH] Raise the number of FD on WIN32/CYGWIN as well

2015-08-26 Thread Olivier Fourdan
Commit 2c94cdb raised the number of file descriptors from 256 to 512 but only for non WIN32/CYGWIN platforms. The definition of XFD_SETSIZE being duplicated, we need to do the same for WIN32/CYGWIN builds as well otherwise a newer X server trying to use more than 256 clients would fail on these pl