Date: Sat, 13 Jan 2007 14:56:18 +0100
From: Bernhard Fischer <rep.dot.nop@gmail.com>
Subject: Re: [Qemu-devel] [PATCH] -parallel and -serial on Windows host -
	second try
Message-ID: <20070113135618.GG14212@aon.at>
References: <eo8nv7$29t$1@sea.gmane.org> <20070113105745.GE14212@aon.at>
	<Pine.LNX.4.63.0701131223220.22628@wbgn013.biozentrum.uni-wuerzburg.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Pine.LNX.4.63.0701131223220.22628@wbgn013.biozentrum.uni-wuerzburg.de>
Reply-To: qemu-devel@nongnu.org
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: qemu-devel@nongnu.org

On Sat, Jan 13, 2007 at 12:28:26PM +0100, Johannes Schindelin wrote:
>Hi,
>
>On Sat, 13 Jan 2007, Bernhard Fischer wrote:
>
>> On Fri, Jan 12, 2007 at 11:36:06AM -0800, alex wrote:
>> 
>> >
>> >--- /d/qemu/vl.c Fri Jan 12 10:43:12 2007
>> >+++ vl.c Fri Jan 12 10:49:37 2007
>> >@@ -2692,8 +2692,13 @@
>> >             if (ret < 0) {
>> >                 err = socket_error();
>> >                 if (err == EINTR || err == EWOULDBLOCK) {
>> >+#ifndef _WIN32
>> >                 } else if (err == EINPROGRESS) {
>> >                     break;
>> >+#else
>> >+                } else if (err == EINPROGRESS | err == WSAEALREADY) {
>> 
>> Doesn't sound like it would be correct ("|" != "||"), even for Windows.. 
>
>Well, it might be not intended, but it still works correctly: "==" has a 
>higher priority than "|", and the value of "err == EINPROGRESS" is 0 or 1, 
>depending on the value of EINPROGRESS. It is never anything else but 0 or 
>1, so the operator "|" has the same result as "||" (at least in this 
>case).

Your right, sorry for the noise.