Messages in this thread |  | | | Date | Tue, 21 Dec 2010 12:51:10 +0000 | | From | Andy Whitcroft <> | | Subject | Re: [PATCH][RESEND] scripts: add no-warning option to the checkpatch script |
| |
On Mon, Dec 20, 2010 at 04:06:10AM -0800, Joe Perches wrote: > On Mon, 2010-12-20 at 12:48 +0100, Marco Stornelli wrote: > > 2010/12/20 Joe Perches <joe@perches.com>: > > > On Mon, 2010-12-20 at 09:37 +0100, Marco Stornelli wrote: > > >> 2010/12/20 Joe Perches <joe@perches.com>: > > >> > Why do you think it's useful? > > >> To "debug" a patch from style point of view, I think it's useful to > > >> fix before the errors and then the warnings. So this option it's > > >> useful as a "filter". However, I added you in cc: because of a reply > > >> to my first patch that indicated you as a person interested in this > > >> scope. > > > Perhaps then it'd be better to filter/enable/disable > > > the 3 current categories of messages. > > > Maybe something like this: > > Yeah, very good. Will it be inserted in mainline? > > Dunno. Maybe if Andy thinks it worthwhile. > He takes his time about this sort of thing.
At a quick skim that seems a more comprehensive approach. I've been lax as always picking things up but with the quiet time coming I hope to hoover up the left overs and push them out.
Thanks.
-apw
|  |