wineconsole: Wait on events in curses backend using select instead of poll

[Kirill K. Smirnov]

>Should I add a 
>check for sys/select.h in the configure script and conditionally included 
>it, surrounded by HAVE_SYS_SELECT_H? Or would sys/time.h, sys/types.h and 
>unistd.h be enough to consider it reasonably compatible?
Ideally - should. But...
1) There isn't any 'select.h' substring in whole wine tree.
2) There is a file libs/port/usleep.c which uses select() but it does not 
include any meaningful headers except "config.h" and "wine/port.h". Are these 
two headers enough wineconsole - I have no idea. Alexandre knows better ;-) 
Try and see ;-)

>After fixing this, I guess I should just submit a new patch against the 
>current upstream git (not against the old patched version), with the same 
>commit message, replacing the old one?
Yes, and mark a new mail subject with tag '[try 2]' to easily distinguish 
between different version of the same fix.

--
Kirill

wineconsole: Wait on events in curses backend using select instead of poll

[Martin Storsjö]

On Sun, 30 Nov 2008, Kirill K. Smirnov wrote:

>> Should I add a
>> check for sys/select.h in the configure script and conditionally included
>> it, surrounded by HAVE_SYS_SELECT_H? Or would sys/time.h, sys/types.h and
>> unistd.h be enough to consider it reasonably compatible?
> Ideally - should. But...
> 1) There isn't any 'select.h' substring in whole wine tree.
> 2) There is a file libs/port/usleep.c which uses select() but it does not
> include any meaningful headers except "config.h" and "wine/port.h". Are these
> two headers enough wineconsole - I have no idea. Alexandre knows better ;-)
> Try and see ;-)

Ok, thanks!

> Yes, and mark a new mail subject with tag '[try 2]' to easily distinguish
> between different version of the same fix.

Sigh, I somehow managed to forget adding this when mailing the second 
attempt. Oh well...

// Martin