On 2014/06/16, 11:43 PM, "Anil Belur" <ask...@gmail.com> wrote:
>From: Anil Belur <ask...@gmail.com> > >fixed: WARNING: suspect code indent for conditional statements (32, 32) >Signed-off-by: Anil Belur <ask...@gmail.com> >--- > drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >diff --git a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c >b/drivers/staging/lustre/lustre/lclient/lcommon_cl.c >index e2da397..3b2b995 100644 >--- a/drivers/staging/lustre/lustre/lclient/lcommon_cl.c >+++ b/drivers/staging/lustre/lustre/lclient/lcommon_cl.c >@@ -825,8 +825,8 @@ int ccc_prep_size(const struct lu_env *env, struct >cl_object *obj, > loff_t size_index = ((size - 1) >> > PAGE_CACHE_SHIFT); > > if ((size == 0 && cur_index != 0) || >- (size_index < cur_index)) >- *exceed = 1; >+ (size_index < cur_index)) The old indentation of this line (aligned after '(' on the previous line) was better. Note also that there is no need for parenthesis around this statement. >+ *exceed = 1; > } > return result; > } else { >-- >1.9.0 > > Cheers, Andreas -- Andreas Dilger Lustre Software Architect Intel High Performance Data Division -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/