* Ingo Molnar <[email protected]> wrote:
> what the hell are you thinking? Not every trivial line of code needs
> to be commented. Comments are needed for the _nontrivial_ lines of
> code, and there's no way a tool can decide that. [...]
and i thought you understood this point, as earlier versions of
checkpatch.pl were carefully filtering out _known nontrivial_ code, such
as the barrier APIs (smp_rmb(), etc.) where we know that they are
nontrivial in _100%_ of the cases.
trying to warn about code that 'might' be unclean is a catastrophy. This
tool should only make noise if it is _really_ sure that there's
something wrong going on. If the tool cannot make an intelligent
decision about it then it should be silent!
Btw., there's an easy solution for v0.08's many breakages: turn off the
WARNING output by default and rename it to 'INFO' or something other
non-alarming.
Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
