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

2011-11-12 Thread David Chisnall
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 versi

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

2011-11-13 Thread Peter Jeremy
On 2011-Nov-12 20:16:07 +, David Chisnall wrote: > Fix SIGATOMIC_M{IN,AX} on x86-64. Thanks for that. I have two comments: 1) Could you please wrap your log messages so that lines do not exceed 80 characters. > /* Limits of sig_atomic_t. */ >-#define SIG_ATOMIC_MIN INT32_MIN >-#def

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

2011-11-14 Thread Dimitry Andric
On 2011-11-14 06:31, Peter Jeremy wrote: > On 2011-Nov-12 20:16:07 +, David Chisnall wrote: ... >> +#define SIG_ATOMIC_MIN LONG_MIN >> +#define SIG_ATOMIC_MAX LONG_MAX > > 2) As far as I can see, all other defines in _stdint.h, across all > architectures, have explicit lengths, so I