[U-Boot] Policy for checkpatch usage?
Eric Cooper
ecc at cmu.edu
Thu Apr 21 17:04:07 CEST 2011
On Thu, Apr 21, 2011 at 04:56:36PM +0200, Fabian Cenedese wrote:
> Is that even possible? The cosmetic patch itself will be surrounded
> by context lines which may fire up a warning. So these lines need
> to be changed as well to satisy checkpatch. But this new patch
> will again include several context lines... until you have to fix up the
> whole file. Or did I misunderstand?
What's wrong with (cosmetically) fixing all the files that a patch
touches? That way the project gets incremental cleanup of the code
base as it evolves.
(It would be easy to automate a check for whitespace-only patches to
ease the job of the custodians. Line-breaking and other style changes
might still require eyeballing.)
--
Eric Cooper e c c @ c m u . e d u
More information about the U-Boot
mailing list