diff mbox

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

Message ID 63DD81B3-3848-4449-A7BA-50B294D7FF19@adacore.com
State New
Headers show

Commit Message

Tristan Gingold March 9, 2012, 1:05 p.m. UTC 
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.

Comments

Joseph Myers March 9, 2012, 5:22 p.m. UTC | #1 
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.
diff mbox

Patch

diff --git a/gcc/ginclude/stddef.h b/gcc/ginclude/stddef.h
index 8a03948e..ebe0190 100644
--- a/gcc/ginclude/stddef.h
+++ b/gcc/ginclude/stddef.h
@@ -203,6 +203,8 @@  typedef __PTRDIFF_TYPE__ ptrdiff_t;
 #if (defined (__FreeBSD__) && (__FreeBSD__ >= 5)) \
   || defined(__FreeBSD_kernel__)
 /* __size_t is a typedef on FreeBSD 5, must not trash it. */
+#elif defined (VMS)
+/* __size_t is also a typedef on VMS.  */
 #else
 #define __size_t
 #endif