Re: [PATCH v3 7/7] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

2014-06-16 Thread Anil Shashikumar Belur
> + unsigned long cur_index; > I wonder why move this particular declaration here? > The only user is still in that one conditional branch anyway. These changes are for fixing warning of line over 80 chars and indent. maybe I should change 'unsigned long' to 'u64' keeping the line of code in

Re: [PATCH v3 7/7] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

2014-06-16 Thread Anil Shashikumar Belur
+ unsigned long cur_index; I wonder why move this particular declaration here? The only user is still in that one conditional branch anyway. These changes are for fixing warning of line over 80 chars and indent. maybe I should change 'unsigned long' to 'u64' keeping the line of code in

Re: [PATCH v3 7/7] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

2014-06-15 Thread Drokin, Oleg
On Jun 16, 2014, at 12:28 AM, Anil Belur wrote: > From: Anil Belur > > fixed: WARNING: line over 80 characters and indent after the conditional > statement > > Signed-off-by: Anil Belur > --- > drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 9 ++--- > 1 file changed, 6

[PATCH v3 7/7] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

2014-06-15 Thread Anil Belur
From: Anil Belur fixed: WARNING: line over 80 characters and indent after the conditional statement Signed-off-by: Anil Belur --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[PATCH v3 7/7] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

2014-06-15 Thread Anil Belur
From: Anil Belur ask...@gmail.com fixed: WARNING: line over 80 characters and indent after the conditional statement Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

Re: [PATCH v3 7/7] staging: lustre: lclient: lcommon_cl.c fixing coding style issues

2014-06-15 Thread Drokin, Oleg
On Jun 16, 2014, at 12:28 AM, Anil Belur wrote: From: Anil Belur ask...@gmail.com fixed: WARNING: line over 80 characters and indent after the conditional statement Signed-off-by: Anil Belur ask...@gmail.com --- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 9 ++--- 1 file