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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
