[VMS/committed]: Do not define __size_t on VMS

Tristan Gingold gingold@adacore.com
Mon Mar 12 08:50:00 GMT 2012


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.
>> 
>> Tristan.
>> 
>> 2012-03-09  Tristan Gingold  <gingold@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  <gingold@adacore.com>

	* ginclude/stddef.h: Adjust previous patch.  Use __VMS__ instead
	of VMS.


Index: gcc/ginclude/stddef.h
===================================================================
--- gcc/ginclude/stddef.h	(revision 185212)
+++ gcc/ginclude/stddef.h	(working copy)
@@ -203,7 +203,7 @@
 #if (defined (__FreeBSD__) && (__FreeBSD__ >= 5)) \
   || defined(__FreeBSD_kernel__)
 /* __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.  */
 #else
 #define __size_t



More information about the Gcc-patches mailing list