Subject: Re: [PATCH 3/5] checkpatch: add a blacklist
Date: Wednesday 7th October 2009 15:11:07 UTC (over 7 years ago)
> Right it's a tool .. However, you should use it and you should follow > it. If for some reason you disagree with the tool you have to give at > least an arguable reason why, not just "It's a guide", "I don't like the > coding style." etc.. Those are perfectly good reasons. > In the case of Steven's code he has an arguable reason why he's not > following checkpatch.. Checkpatch is not very bright, it has no understanding of style beyond playing with pattern regexps. It's a rather dim tool that helps people get work done. (*) When used at random to "validate" submissions to the kernel the result is about as useful as a square wheel on a hovercraft. Alan (*) or as some would have it a rather dim tool used by even dimmer tools to make noise on kernel list.