On Mon, Feb 28, 2011 at 07:27:06PM +0200, Tomas Winkler wrote: > Add first level indentation to easayca_main.c > This created around 300 lines over 80 characters. > Around 100 of straight forward once were fixed in this patch. > The another 200 require more code movement and need to be fixed > later >
For this kind of patch it's better to ignore the over 80 char warnings and send the fixes for that in a separate patch. I've reviewed all five patches and I didn't see any bugs. Some of the JOM() and SAM() messages and normally that kind of thing should be mentioned in the changelog, but those were small changes so *shrug*. regards, dan carpenter _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
