Messages in this thread | | | From | "Simmons, James A." <> | Subject | RE: [lustre-devel] [PATCH 1/3] staging: lustre: checkpatch cleanups for nidstring.c | Date | Sun, 1 Nov 2015 23:07:42 +0000 |
| |
>On Thu, Oct 29, 2015 at 07:28:21PM -0400, James Simmons wrote: >> With nidstring now having the latest fixes we can >> now clean up all the remaining checkpatch errors >> for nidstring.c. > >Please be specific as to exactly what you changed, and break it up into >one-patch-per-thing. And no, "fix all checkpatch errors" is not "one >thing"
Hmm. This makes me think I might be going about this wrong. Instead of doing style changes per file I should be doing one style change per subsystem instead. Unless you prefer doing these style changes on per file base. Perhaps for now I should focus on pushing the fixes that have cumulated and once caught up then finished off the style issues.
| |