Messages in this thread | | | Date | Mon, 28 Jan 2008 14:38:46 +0000 | From | Andy Whitcroft <> | Subject | Re: [PATCH] checkpatch: add filename in the summary |
| |
On Tue, Jan 22, 2008 at 02:20:48PM +0100, Paolo Ciarrocchi wrote: > On Jan 14, 2008 11:23 PM, Paolo Ciarrocchi <paolo.ciarrocchi@gmail.com> wrote: > > Add a filename option (default to 0) > > in order to get the following summary output: > > ./scripts/checkpatch.pl --filename --file ./arch/sparc/kernel/apc.c > > ... > > ... > > ... > > ./arch/sparc/kernel/apc.c --> total: 18 errors, 3 warnings, 186 lines checked
Ok, I've applied a heavily modified version of this to match the current code. The option is now --summary-file to prevent confusion between it and the --file option.
> Hi Andy, > do you plan to document all the command line options that checkpatch > is currently accepting?
Most of the undocumented options are either used in testing or are for uncommon usage. Some work does need doing on the usage tho.
-apw
| |