Re: [PATCH] checkpatch: add filename in the summary

2008-01-28 Thread Andy Whitcroft
On Tue, Jan 22, 2008 at 02:20:48PM +0100, Paolo Ciarrocchi wrote: > On Jan 14, 2008 11:23 PM, Paolo Ciarrocchi <[EMAIL PROTECTED]> 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.

Re: [PATCH] checkpatch: add filename in the summary

2008-01-22 Thread Paolo Ciarrocchi
On Jan 14, 2008 11:23 PM, Paolo Ciarrocchi <[EMAIL PROTECTED]> 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 wa

Re: [PATCH] checkpatch: add filename in the summary

2008-01-15 Thread Cyrill Gorcunov
[Paolo Ciarrocchi - Tue, Jan 15, 2008 at 10:13:43PM +0330] | On 1/15/08, Cyrill Gorcunov <[EMAIL PROTECTED]> wrote: | > [Paolo Ciarrocchi - Mon, Jan 14, 2008 at 11:23:39PM +0100] | | > Hi Paolo, | > | > the options you're complaining about (in most) are enabled by default so | > there is no reason

Re: [PATCH] checkpatch: add filename in the summary

2008-01-15 Thread Paolo Ciarrocchi
On 1/15/08, Cyrill Gorcunov <[EMAIL PROTECTED]> wrote: > [Paolo Ciarrocchi - Mon, Jan 14, 2008 at 11:23:39PM +0100] > Hi Paolo, > > the options you're complaining about (in most) are enabled by default so > there is no reason to 'turn them on' explicitly /methink/. > I was looking for more infor

Re: [PATCH] checkpatch: add filename in the summary

2008-01-15 Thread Cyrill Gorcunov
[Paolo Ciarrocchi - Mon, Jan 14, 2008 at 11:23:39PM +0100] | 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, 18