On Mar 9, 2012, at 6:22 PM, Joseph S. Myers wrote:
> On Fri, 9 Mar 2012, Tristan Gingold wrote:
>> as it is already defined and used by the system headers.
>> No regressions for x86_64-darwin after full bootstrap.
>> 2012-03-09 Tristan Gingold <ging...@adacore.com>
>> * ginclude/stddef.h: Do not define __size_t on VMS.
> Sorry, this patch is wrong. The identifier "VMS" is in the user's
> namespace and users on non-VMS platforms must be able to define it as a
> macro without affecting how stddef.h behaves. Please change this to test
> a macro in the implementation namespace instead.
Thank you for catching that. Fixed with:
2012-03-12 Tristan Gingold <ging...@adacore.com>
* ginclude/stddef.h: Adjust previous patch. Use __VMS__ instead
--- gcc/ginclude/stddef.h (revision 185212)
+++ gcc/ginclude/stddef.h (working copy)
@@ -203,7 +203,7 @@
#if (defined (__FreeBSD__) && (__FreeBSD__ >= 5)) \
/* __size_t is a typedef on FreeBSD 5, must not trash it. */
-#elif defined (VMS)
+#elif defined (__VMS__)
/* __size_t is also a typedef on VMS. */