From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:38349)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <revol@free.fr>) id 1YaYc4-0001Ag-Ow
	for qemu-devel@nongnu.org; Tue, 24 Mar 2015 19:52:01 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <revol@free.fr>) id 1YaYc1-0007y6-Iw
	for qemu-devel@nongnu.org; Tue, 24 Mar 2015 19:52:00 -0400
Received: from smtp5-g21.free.fr ([212.27.42.5]:23633)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <revol@free.fr>) id 1YaYc1-0007xk-D6
	for qemu-devel@nongnu.org; Tue, 24 Mar 2015 19:51:57 -0400
Message-ID: <5511F897.6000004@free.fr>
Date: Wed, 25 Mar 2015 00:51:51 +0100
From: =?UTF-8?B?RnJhbsOnb2lzIFJldm9s?= <revol@free.fr>
MIME-Version: 1.0
References: <1427153788-2738-1-git-send-email-kallisti5@unixzen.com>	<5511E4A7.4050407@suse.de>	<CAMo8BfKYhRRthT7rgO3eBGZ7K4_QBoeFKM5-bGJL0v_0gFgtmg@mail.gmail.com>	<5511ED48.7070107@free.fr>
	<CAMo8BfLTkmV1Lt7DhDedO7kyNdz8JjB9nLC-h34S3YRkrsp8Bg@mail.gmail.com>
	<5511F5E5.8000805@suse.de>
In-Reply-To: <5511F5E5.8000805@suse.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH] Haiku: Platform build fixes
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= <afaerber@suse.de>
Cc: Max Filippov <jcmvbkbc@gmail.com>, Alexander von Gluck IV <kallisti5@unixzen.com>, qemu-devel <qemu-devel@nongnu.org>

On 25/03/2015 00:40, Andreas F=C3=A4rber wrote:
> Am 25.03.2015 um 00:11 schrieb Max Filippov:
>> On Wed, Mar 25, 2015 at 2:03 AM, Fran=C3=A7ois Revol <revol@free.fr> w=
rote:
>>> On 24/03/2015 23:56, Max Filippov wrote:
>>>> On Wed, Mar 25, 2015 at 1:26 AM, Andreas F=C3=A4rber <afaerber@suse.=
de> wrote:
>>>>> Am 24.03.2015 um 00:36 schrieb Alexander von Gluck IV:
>>>>>>
>>>>>
>>>>> Why convert from array to switch statement? It looks like a very
>>>>> invasive change for no obvious reason.
>>>>
>>>> I'd be interested to know the reason too, but I'm OK with either way=
.
>>>
>>> Maybe because negative array indices are usually a bad idea in C?
>>
>> Sounds like a good reason (:
>=20
> True, but assumes that error codes are indeed negative.
>=20
> AFAIU linking with -lposix_error_helper or so (-> configure/Makefile*)
> may be an alternative, keeping errors POSIX-compliant in our code. I
> don't recall the implementation details though...

Indeed, although it doesn't solve everything (some things like nginx
want to use deprecated things like sys_errlist[] (although they dropped
this) or assume the codes are all < some very low positive value), and I
suspect it could have strange side effects on code used as a library by
other things.

As for myself, I consider POSIX to be boggus (they even changed their
mind in some places) and error prone on this (I've ported enough foreign
code to assert this) :p

Fran=C3=A7ois.