On Sat, 2014-07-12 at 12:28 -0700, Greg KH wrote: > On Sat, Jul 12, 2014 at 11:29:37AM -0700, Joe Perches wrote: > > On Sat, 2014-07-12 at 10:55 -0700, Greg KH wrote: > > > Yes, the warnings are all due to line-length, but Joe, you shouldn't add > > > a patch that causes more checkpatch warnings than before :) > > > > Yeah, that was intentional though. > > > > This script does a series of relatively discrete changes. > > > > Lindent would more or less work, but it's _horrible_ at > > wrapping overlong lines and merges all types of changes > > together. > > Oh I agree, I don't want to see Lindent, but maybe, if the patch adds > checkpatch warnings, it should be at least "flagged" as maybe a problem? The script I submitted does run checkpatch before any commit is done. > Would you want to be on the receiving end of this patch script? You might have noticed I don't have a kernel.org account. _______________________________________________ Kernelnewbies mailing list Kernelnewbies@xxxxxxxxxxxxxxxxx http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
