NetBSD-Bugs archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: kern/43200: Remove trailing comma in rump.h.
On Sat, Apr 24, 2010 at 05:55:50AM +0200, Martin Husemann wrote:
> On Fri, Apr 23, 2010 at 07:03:18PM -0400, Christos Zoulas wrote:
> > Not in the current version of c++ though I think. Eventually but not now.
>
> They have been always allowed in C++.
> Trailing comma is prefered style, so what is this PR about, after all?
While I have no strong opinion, we need to remember that rump.h
should be buildable on a non-NetBSD toolchain. That said, you need
a C99 compiler for the NetBSD kernel, so situations where this has
any practical relevance are likely far and apart.
And, actually, trailing comma for enums is not preferred style (+).
But I propose we change it -- not getting diffs for the previous
line when appending an enum is useful.
+)
/* $NetBSD: style,v 1.46 2010/04/24 00:54:07 christos Exp $ */
[...]
/* Enum types are capitalized. No comma on the last element. */
enum enumtype {
ONE,
TWO
} et;
Home |
Main Index |
Thread Index |
Old Index