Re: [PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-17 Thread Greg Kroah-Hartman
On Sat, Jul 15, 2017 at 11:39:22AM -0400, James Simmons wrote: > Cleanup all braces that was reported by checkpatch. The only > issue not fixed up is in mdc_lock.c. Removing the braces in > the case of mdc_lock.c will break the build. > > Signed-off-by: James Simmons > ---

Re: [PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread James Simmons
> On Sat, 2017-07-15 at 11:39 -0400, James Simmons wrote: > > Cleanup all braces that was reported by checkpatch. The only > > issue not fixed up is in mdc_lock.c. Removing the braces in > > the case of mdc_lock.c will break the build. > > what checkpatch warning in mdc_locks.c is that? > > $ ./

Re: [PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread Joe Perches
On Sat, 2017-07-15 at 11:39 -0400, James Simmons wrote: > Cleanup all braces that was reported by checkpatch. The only > issue not fixed up is in mdc_lock.c. Removing the braces in > the case of mdc_lock.c will break the build. what checkpatch warning in mdc_locks.c is that? $ ./scripts/checkpatc

[PATCH] staging: lustre: lustre: fix all braces issues reported by checkpatch

2017-07-15 Thread James Simmons
Cleanup all braces that was reported by checkpatch. The only issue not fixed up is in mdc_lock.c. Removing the braces in the case of mdc_lock.c will break the build. Signed-off-by: James Simmons -- v1) Initial patch v2) Rebased again