[1/2] server: fix async read on mailslot

Andrey Turkin andrey.turkin at gmail.com
Fri Jul 18 11:37:18 CDT 2008


Hi Ambroz,

> -                                            
> FILE_SYNCHRONOUS_IO_NONALERT );
> +                                            read_timeout ? 0 : 
> FILE_SYNCHRONOUS_IO_NONALERT );
This looks incorrect. You probably want to compare to -1, not zero. Zero 
means reads must not block at all, and -1 means read must wait forever 
(not sure what zero timeout would work as expected with your patch, but 
it didn't anyway and I think it maybe is broken in another place)
Also, please attach patches with disposition inline, not attachment (git 
can do that) - this make review somehow easier.

Regards,
  Andrey


Ambroz Bizjak wrote:
> Hi,
>
> Changelog:
> server: fix async read on mailslot
> ------------------------------------------------------------------------
>
>
>   

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-patches/attachments/20080718/46404e38/attachment.htm 


More information about the wine-patches mailing list