On 7/24/07, Adrian Bunk <bunk@stusta.de> wrote:
There are more important things than exploiting the corner cases of codingstyle, e.g. could you teach checkpatch.pl to give exactly two errors for the following code? while (a); for (b = 0; b < 50; b++); for (c = 0; c < sizeof(struct module); c++) d = e;
There are three errors there. The while (a) busy wait needs a cpu_relax() or something, the first for is at the wrong level of indentation and the second for is at the wrong level of indentation relative to the first one. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Adrian Bunk <bunk@stusta.de>
- Re: [PATCH] update checkpatch.pl to version 0.08
- References:
- [PATCH] update checkpatch.pl to version 0.08
- From: Andy Whitcroft <apw@shadowen.org>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: "Kok, Auke" <auke-jan.h.kok@intel.com>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Andy Whitcroft <apw@shadowen.org>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Andrew Morton <akpm@linux-foundation.org>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Paul Mundt <lethal@linux-sh.org>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Jan Engelhardt <jengelh@computergmbh.de>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Andy Whitcroft <apw@shadowen.org>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Adrian Bunk <bunk@stusta.de>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: jschopp <jschopp@austin.ibm.com>
- Re: [PATCH] update checkpatch.pl to version 0.08
- From: Adrian Bunk <bunk@stusta.de>
- [PATCH] update checkpatch.pl to version 0.08
- Prev by Date: Re: [PATCH 0/3] readahead drop behind and size adjustment
- Next by Date: Re: [patch] fix the softlockup watchdog to actually work
- Previous by thread: Re: [PATCH] update checkpatch.pl to version 0.08
- Next by thread: Re: [PATCH] update checkpatch.pl to version 0.08
- Index(es):
![]() |