aboutsummaryrefslogtreecommitdiffstats
path: root/Documentation
diff options
context:
space:
mode:
authorAndy Whitcroft <apw@shadowen.org>2007-07-16 02:37:22 -0400
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2007-07-16 12:05:34 -0400
commitde7d4f0e1172a72277d660fa0be59654ea02bed0 (patch)
tree6774ab290e4bb257a6207005f7072788895cce05 /Documentation
parentf2a11b158a24301e9158e9c873fa88e5eb775486 (diff)
update checkpatch.pl to version 0.07
This version brings a number of new checks, fixes for flase positives, plus a clarification of the output to better guide use. Of note: - checks for documentation for new __setup calls - clearer reporting where braces and parenthesis are involved - reports for closing brace and semi-colon spacing - reports on unwanted externs This patch includes an update to the documentation on checkpatch.pl itself to clarify when it should be used and to indicate that it is not intended as the final arbitor of style. Full changelog: Andy Whitcroft (19): Version: 0.07 ensure we do not apply control brace checks to preprocesor directives add {u,s}{8,16,32,64} to the type matcher accept lack of spacing after the semicolons in for (;;) report new externs in .c files fix up typedef exclusion for function prototypes else trailing statements check need to account for \ at end of line add enums to the type matcher add missing check descriptions suppress double reporting of ** spacing report on do{ spacing issues include an example of the brace/parenthesis in output check for spacing after closing braces prevent double reports on pointer spacing issues handle blank continuation lines on macros classify all reports error, warning, or check revamp hanging { checks and apply in context no spaces after the last ; in a for is ok check __setup has a corresponding addition to documentation David Woodhouse (1): limit character set used in patches and descriptions to UTF-8 Signed-off-by: Andy Whitcroft <apw@shadowen.org> Cc: David Woodhouse <dwmw2@infradead.org> Cc: "Randy.Dunlap" <rdunlap@xenotime.net> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'Documentation')
-rw-r--r--Documentation/SubmittingPatches20
1 files changed, 18 insertions, 2 deletions
diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index 0958e97d4bf4..3f9a7912e69b 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -464,9 +464,25 @@ section Linus Computer Science 101.
464Nuff said. If your code deviates too much from this, it is likely 464Nuff said. If your code deviates too much from this, it is likely
465to be rejected without further review, and without comment. 465to be rejected without further review, and without comment.
466 466
467Once significant exception is when moving code from one file to
468another in this case you should not modify the moved code at all in
469the same patch which moves it. This clearly delineates the act of
470moving the code and your changes. This greatly aids review of the
471actual differences and allows tools to better track the history of
472the code itself.
473
467Check your patches with the patch style checker prior to submission 474Check your patches with the patch style checker prior to submission
468(scripts/checkpatch.pl). You should be able to justify all 475(scripts/checkpatch.pl). The style checker should be viewed as
469violations that remain in your patch. 476a guide not as the final word. If your code looks better with
477a violation then its probably best left alone.
478
479The checker reports at three levels:
480 - ERROR: things that are very likely to be wrong
481 - WARNING: things requiring careful review
482 - CHECK: things requiring thought
483
484You should be able to justify all violations that remain in your
485patch.
470 486
471 487
472 488