<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
  <title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Hi Ambroz,<br>
<br>
<blockquote type="cite">-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
FILE_SYNCHRONOUS_IO_NONALERT );<br>
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; read_timeout ? 0 :
FILE_SYNCHRONOUS_IO_NONALERT );<br>
</blockquote>
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)<br>
Also, please attach patches with disposition inline, not attachment
(git can do that) - this make review somehow easier.<br>
<br>
Regards,<br>
&nbsp; Andrey<br>
<br>
<br>
Ambroz Bizjak wrote:
<blockquote
 cite="mid:56214.193.77.101.149.1216392091.squirrel@webmail.b4ever.net"
 type="cite">
  <pre wrap="">Hi,

Changelog:
server: fix async read on mailslot</pre>
  <pre wrap="">
<hr size="4" width="90%">

  </pre>
</blockquote>
<br>
</body>
</html>