Re: [PATCH] checkpatch: allow single space before labels

From: Peter Zijlstra
Date: Mon Oct 18 2010 - 11:04:11 EST


On Mon, 2010-10-18 at 16:58 +0200, Peter Zijlstra wrote:
> On Mon, 2010-10-18 at 10:54 -0400, Ted Ts'o wrote:
> > Note that there is absolutely nothing about how labels should be
> > indented in CodingStyle at all, and I very much resist code
> > straightjackets being imposed by checkpatch.pl implementors.
> >
> > Can we please stop this nonsense?
>
> 1) ignoring checkpatch over taste is good
> 2) by 1) its impossible to get checkpatch right
> 3) therefore pure checkpatch patches are nonsense
>
> So can we agree that checkpatch is ok in its current form, and we should
> simply administer physical violence towards checkpatch wankers?
>
> PS. I frequently practise 2).

That should be 1) of course.. I frequently ignore checkpatch output.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/