Re: svn commit: r286168 - head/sys/net

2015-08-02 Thread Bjoern A. Zeeb
On 02 Aug 2015, at 00:15 , John-Mark Gurney j...@freebsd.org wrote: -CTASSERT(sizeof(struct sadb_x_policy) == 16); +_Static_assert(sizeof(struct sadb_x_policy) == 16, struct size mismatch); If this fires, how does it look like? I am assuming the string at the end is the error message? If

Re: svn commit: r286168 - head/sys/net

2015-08-02 Thread David Chisnall
On 2 Aug 2015, at 17:34, Ian Lepore i...@freebsd.org wrote: It generates a compiler error, so the output is going to contain file-and-line like any other compiler error, as well as the message from the source code. It will, of course, vary between compilers, but this is what clang generates:

Re: svn commit: r286168 - head/sys/net

2015-08-02 Thread Ian Lepore
On Sun, 2015-08-02 at 11:42 +, Bjoern A. Zeeb wrote: On 02 Aug 2015, at 00:15 , John-Mark Gurney j...@freebsd.org wrote: -CTASSERT(sizeof(struct sadb_x_policy) == 16); +_Static_assert(sizeof(struct sadb_x_policy) == 16, struct size mismatch); If this fires, how does it look like? I

Re: svn commit: r286168 - head/sys/net

2015-08-01 Thread John-Mark Gurney
Bruce Evans wrote this message on Sun, Aug 02, 2015 at 14:10 +1000: On Sun, 2 Aug 2015, John-Mark Gurney wrote: Log: convert to C11's _Static_assert, and pull in sys/cdefs.h for compatibility w/ older non-C11 compilers... This include is bogus. net/pfkeyv2.h already depends on the

Re: svn commit: r286168 - head/sys/net

2015-08-01 Thread Bruce Evans
On Sun, 2 Aug 2015, John-Mark Gurney wrote: Log: convert to C11's _Static_assert, and pull in sys/cdefs.h for compatibility w/ older non-C11 compilers... This include is bogus. net/pfkeyv2.h already depends on the includer including other headers that include sys/cdefs.h. Mainly

svn commit: r286168 - head/sys/net

2015-08-01 Thread John-Mark Gurney
Author: jmg Date: Sun Aug 2 00:15:52 2015 New Revision: 286168 URL: https://svnweb.freebsd.org/changeset/base/286168 Log: convert to C11's _Static_assert, and pull in sys/cdefs.h for compatibility w/ older non-C11 compilers... passed make tinerdbox.. Suggested by: imp Modified: