[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20101221125110.GI18715@shadowen.org>
Date: Tue, 21 Dec 2010 12:51:10 +0000
From: Andy Whitcroft <apw@...onical.com>
To: Joe Perches <joe@...ches.com>
Cc: Marco Stornelli <marco.stornelli@...il.com>,
Linux Kernel <linux-kernel@...r.kernel.org>
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@...ches.com>:
> > > On Mon, 2010-12-20 at 09:37 +0100, Marco Stornelli wrote:
> > >> 2010/12/20 Joe Perches <joe@...ches.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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists