svn commit: r227474 - head/sys/amd64/include

David Chisnall theraven at FreeBSD.org
Sat Nov 12 20:16:07 UTC 2011


Author: theraven
Date: Sat Nov 12 20:16:06 2011
New Revision: 227474
URL: http://svn.freebsd.org/changeset/base/227474

Log:
  Fix SIGATOMIC_M{IN,AX} on x86-64.  These are meant to be the minimum values that are allowed in a sig_atomic_t, but it looks like they were just copied from the x86 versions, so these definitions violate the C and C++ specs.  Mismatch was spotted by the libc++ test suite.
  
  Approved by:	dim (mentor)

Modified:
  head/sys/amd64/include/_stdint.h

Modified: head/sys/amd64/include/_stdint.h
==============================================================================
--- head/sys/amd64/include/_stdint.h	Sat Nov 12 20:01:30 2011	(r227473)
+++ head/sys/amd64/include/_stdint.h	Sat Nov 12 20:16:06 2011	(r227474)
@@ -150,8 +150,8 @@
 #define	PTRDIFF_MAX	INT64_MAX
 
 /* Limits of sig_atomic_t. */
-#define	SIG_ATOMIC_MIN	INT32_MIN
-#define	SIG_ATOMIC_MAX	INT32_MAX
+#define	SIG_ATOMIC_MIN	LONG_MIN
+#define	SIG_ATOMIC_MAX	LONG_MAX
 
 /* Limit of size_t. */
 #define	SIZE_MAX	UINT64_MAX


More information about the svn-src-all mailing list