Messages in this thread | | | Date | Mon, 14 Jan 2008 17:30:32 +0000 | From | Andy Whitcroft <> | Subject | Re: checkpatch: add filename before the summary line |
| |
On Sun, Jan 13, 2008 at 11:34:05PM +0100, Paolo Ciarrocchi wrote: > With the patch applied the output of checkpatch.pl is as follow: > ./arch/arm/mach-footbridge/ebsa285-pci.c > total: 1 errors, 0 warnings, 48 lines checked > > Adding the file name allowed me to collects stats running: > find . -name *.c |xargs ./scripts/checkpatch.pl --file |grep -B 1 total > but I though it might me useful for other purposes to add the file name > after the list of errors and warnings and before the summary, > when the list is long I find handy to print out the filename as a reminder.
Hmmm, that being unconfitional would probabally break a raft of other users. Also would it be more useful to put it on the front of the summary line? So that if you have a bunch of files to check then you get something more like:
foo: total: 1 errors, ... bar: total: 0 errors, ...
-apw
| |