Hi,

On Sun, Mar 25, 2012 at 12:51:55PM +0200, Fabian Knittel wrote:
> 2012/3/24 Alon Bar-Lev <alon.bar...@gmail.com>:
> > On Sat, Mar 24, 2012 at 9:15 PM, Alon Bar-Lev <alon.bar...@gmail.com> wrote:
> >> PPC architectures and conflict with stdbool.h.
> >>
> >> basic.h defines a type 'bool' that conflicts with the altivec
> >> keyword bool which has to be fixed upstream, see bugs[1][2].
> >>
> >> [1] https://bugs.gentoo.org/show_bug.cgi?id=293840
> >> [2] https://bugs.gentoo.org/show_bug.cgi?id=297854
> 
> The approach attempted in [2], using stdbool.h, looks far less
> intrusive than renaming "bool" to "obool" everywhere, so why did you
> choose the latter approach? Jan Scholz's comment #6 [3] sounds like
> the right approach to me ...

The problem is that stdbool.h seems to define a data type that is not
necessarily what the rest of OpenVPN expects (structure sizes etc),
which is also mentioned in these two bug reports - read: it will *break*
OpenVPN, or at least the plugin interface.  So moving to stdbool.h
would require careful analysis of every single usage of "bool" to see
whether this is really only doing "boolean" stuff, or whether there's
other issues lurking here (causing the breakage David noticed in [2]).


Personally I think that this is a regression in GCC, and we should NOT
do anything about it.  If the compiler is documented to have a new
data type "bool__", it should not mess with "bool", which is a different
name by all means - bad enough that we have to jump through hoops to
care for MSVC, but that's at least a mainstream platform, as opposed
to "PPC with a specific compiler option (-maltivec)".

But I'm not insisting on this either way.  This is personal optionion,
and stuff like that is close to coding style and religion anyway.

gert
-- 
USENET is *not* the non-clickable part of WWW!
                                                           //www.muc.de/~gert/
Gert Doering - Munich, Germany                             g...@greenie.muc.de
fax: +49-89-35655025                        g...@net.informatik.tu-muenchen.de

Attachment: pgp7S_CALOJrr.pgp
Description: PGP signature

Reply via email to