lkml.org 
[lkml]   [2007]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] update checkpatch.pl to version 0.10
> The only question is whether this should default to on.  You are voting
> off. I personally think on.
>
> Andrew? Randy? Joel?

The main audience of this is new contributors, who should have more verbose
output, including nitpicky things like multiple assignments per line. The
default should target them. More advanced users can certainly use a flag
that says "give me only the real errors".

It might be a good idea to have three levels.
--really-errors
--really-picky
--really-experimental

Only with better names. --really-picky would be default, but would only
include tests that have a very very high ratio of hits to false positives,
but would still call out things like multiple assignments per line.
--really-errors we could call the Igno level. --really-experimental would
call out all issues, even on checks that generate a fair number of false
positives.

-Joel

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-09-28 17:53    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site