[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] checkpatch: remove rule on non-indented labels
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] checkpatch: remove rule on non-indented labels |
Date: |
Fri, 7 Oct 2011 15:43:10 +0100 |
On 7 October 2011 14:59, Paolo Bonzini <address@hidden> wrote:
> There are 508 non-indented (non-default) labels, and 511 that are
> indented. So the rule is debatable at least. Actually, in the
> common case of labels at the outermost scope, there is really just
> one place where to put the label, so the rule is just wrong IMHO.
Agreed. This is one of the checkpatch shibboleths that I just
ignore when submitting patches :-)
-- PMM