Subject: Re: CVS commit: src/sys/sys (protosw.h:1.39)
To: None <source-changes@NetBSD.org>
From: Geoff Wing <gcw@pobox.com>
List: source-changes
Date: 08/29/2006 08:49:49
On Monday 2006-08-28 17:26 +0400, Valeriy E. Ushakov output:
:On Mon, Aug 28, 2006 at 13:37:41 +1000, Geoff Wing wrote:
:> Yeah, I only had a problem with netinet/in_proto.c when compiling a
:> kernel but I listed all with ones with empty trailing elements in
:> the struct assignments,
:> i.e. { a, b, c, }, ... vs { a, b, c }, ...
:It's not an empty trailing element. C99 allows the comma after the
:alst element in the list of initiliazers (and also in enums).
I guess most of my C is still back in the 80s with only small bits and pieces
from recent era :-) Thank you for the explanation. Possibly I've come
across it before and just forgotten.
In this case a struct element was removed so (in a very simple form):
struct { int a, b, c; } foo[] = { { 1, 2, } };
went to
struct { int a, b; } foo[] = { { 1, 2, } };
with the structure changing but the initializer remaining the same.
I thought it somewhat in poor form.
In another message, Christos said that they'll be fixed later. Since it
compiles properly now, I presume it's not high priority.
Regards,
Geoff