Changeset 63491 in vbox

Timestamp:

Aug 15, 2016 4:41:16 PM (8 years ago)

Author:

vboxsync

Message:

iprt/types.h: On NetBSD after we include <stdbool.h> undef "bool" it
defines and provide a typedef instead. This is necessary because some
sloppy X11 code uses "bool" as a structure member name.

The end result is the same as for Darwin and Haiku clause just below,
but I think that explicit #include <stdbool.h> here (vs. checking for
its guard) is more clean, since it ensures that any code that includes
<stdbool.h> later will not cause additionall damage.

File:

• trunk/include/iprt/types.h (modified) (1 diff)

trunk/include/iprt/types.h

@@ -239 +239 @@
       */
 #    include <stdbool.h>
+
+     /*
+      * ... but the story doesn't end here.  The C standard says that
+      * <stdbool.h> defines preprocessor macro "bool" that expands to
+      * "_Bool", but adds that a program may undefine/redefine it
+      * (this is 7.16 in C99 and 7.18 in C11).  We have to play this
+      * game here because X11 code uses "bool" as a struct member name
+      * - so undefine "bool" and provide it as a typedef instead.  We
+      * still keep #include <stdbool.h> so that any code that might
+      * include it later doesn't mess things up.
+      */
+#    undef bool
+     typedef _Bool bool; 
 #   endif
 #  else