From: Erez Zadok <ezk@cs.sunysb.edu>
To: apw@shadowen.org, rdunlap@xenotime.net, jschopp@austin.ibm.com
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH] 0/3 checkpatch updates, new checkfiles script
Date: Fri, 5 Oct 2007 12:56:05 -0400 [thread overview]
Message-ID: <11916033681196-git-send-email-ezk@cs.sunysb.edu> (raw)
This series of patches adds a -t option to checkpatch, so it can print terse
messages one per line, in a format compatible with g/cc
(filename:linenumber:message). This format can be parsed by various tools
and editors that can help show the errors in one window and the offending
file+line in another window.
This patch series also introduces a new small shell script wrapper, called
scripts/checkfiles, that checks the compliance of source files (not in diff
format); the script can operate on any mix of files and directories
(recursively checking). For example, to check the entire Linux kernel, run:
$ ./scripts/checkfiles .
So, I ran the above script and it found nearly 1.5 million reported
warnings/errors, with drivers being the largest abuser, not surprisingly.
Here's the sorted breakdown per top-level subsystem:
32 usr
162 init
266 block
293 Documentation
471 ipc
819 mm
1115 security
1915 scripts
1948 kernel
4569 lib
4793 crypto
13851 net
80025 sound
86484 include
115789 arch
116623 fs
1035158 drivers
-------+-------
1464313 TOTAL
Any volunteers? :-)
Erez Zadok (3):
CHECKPATCH: update usage string for checkpatch.pl
CHECKPATCH: add terse output option to checkpatch.pl
CHECKFILES: new small shell script to check multiple source files
checkfiles | 34 ++++++++++++++++++++++++++++++++++
checkpatch.pl | 21 +++++++++++++++++----
2 files changed, 51 insertions(+), 4 deletions(-)
---
Erez Zadok
ezk@cs.sunysb.edu
next reply other threads:[~2007-10-05 17:11 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-05 16:56 Erez Zadok [this message]
2007-10-05 16:56 ` [PATCH 1/3] CHECKPATCH: update usage string for checkpatch.pl Erez Zadok
2007-10-05 16:56 ` [PATCH 2/3] CHECKPATCH: add terse output option to checkpatch.pl Erez Zadok
2007-10-05 16:56 ` [PATCH 3/3] CHECKFILES: new small shell script to check multiple source files Erez Zadok
2007-10-06 11:13 ` [PATCH] 0/3 checkpatch updates, new checkfiles script Ingo Molnar
2007-10-07 19:05 ` Erez Zadok
2007-10-11 13:47 ` Andy Whitcroft
2007-10-13 18:55 ` Erez Zadok
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=11916033681196-git-send-email-ezk@cs.sunysb.edu \
--to=ezk@cs.sunysb.edu \
--cc=apw@shadowen.org \
--cc=jschopp@austin.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@xenotime.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®