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

2015-08-02 Thread David Chisnall
On 2 Aug 2015, at 17:34, Ian Lepore 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: $ cat stati

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 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 as

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

2015-08-02 Thread Bjoern A. Zeeb
> On 02 Aug 2015, at 00:15 , John-Mark Gurney 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 so and if th

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. already depends on the inclu

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. already depends on the includer including other headers that include . Mainly . defined massive namespace pollu

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: h