lkml.org 
[lkml]   [2008]   [Jan]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Deprecate checkpatch.pl --file mode; add warning; add --file-force
On Wed, Jan 16, 2008 at 11:21:21PM +0100, Andi Kleen wrote:

> Deprecate checkpatch.pl --file mode; add warning; add --file-force
>
> As discussed on linux-kernel checkpatch.pl only patches for whole
> files have a significant cost. Better such changes should be only
> done together with other changes. Add a explicit warning about
> this; deprecate --file and add a --file-force instead.
>
> Signed-off-by: Andi Kleen <ak@suse.de>

Cirtainly I can see why we want this
warning out there and very much in the submitters face, it just feels a
little heavy handed to make them change option to get the result.

Would not just always appending this message in --file mode always be
just as effective? Perhaps suppressing it with -q, or with some new "i
know what I am doing" option?

-apw


\
 
 \ /
  Last update: 2008-01-28 16:01    [W:0.084 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site