At 10:49 21.04.2011 -0400, Eric Cooper wrote: >On Thu, Apr 21, 2011 at 04:29:17PM +0200, Detlev Zundel wrote: >> What about the problem with checkpatch errors in current code, i.e. the >> origin of this sentence: >> >> Also warnings produced for context lines (i.e. existing code) rather >> than actual changes can also be ignored. > >How about replacing it with this: > > If you encounter warnings for existing code, not modified by your > patch, consider submitting a separate, cosmetic-only patch -- > clearly described as such -- that *precedes* your substantive > patch.
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? bye Fabi _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
