From: Marcin Slusarz fix coding style, checkpatch warned about: - "braces {} are not necessary for single statement blocks" - "line over 80 characters" - "do not use assignment in if condition" - "use tabs not spaces" Signed-off-by: Marcin Slusarz Cc: Ben Fennema Acked-by: Jan Kara Signed-off-by: Andrew Morton --- fs/udf/dir.c | 46 +++++++++++++++++++++++++++------------------- 1 file changed, 27 insertions(+), 19 deletions(-) diff -puN fs/udf/dir.c~udf-fix-coding-style-of-dirc fs/udf/dir.c --- a/fs/udf/dir.c~udf-fix-coding-style-of-dirc +++ a/fs/udf/dir.c @@ -65,24 +65,26 @@ static int do_udf_readdir(struct inode * if (nf_pos == 0) nf_pos = (udf_ext0_offset(dir) >> 2); - fibh.soffset = fibh.eoffset = (nf_pos & ((dir->i_sb->s_blocksize - 1) >> 2)) << 2; + fibh.soffset = fibh.eoffset = + (nf_pos & ((dir->i_sb->s_blocksize - 1) >> 2)) << 2; bits = dir->i_sb->s_blocksize_bits; - if (UDF_I_ALLOCTYPE(dir) == ICBTAG_FLAG_AD_IN_ICB) { + if (UDF_I_ALLOCTYPE(dir) == ICBTAG_FLAG_AD_IN_ICB) fibh.sbh = fibh.ebh = NULL; - } else if (inode_bmap(dir, nf_pos >> (bits - 2), - &epos, &eloc, &elen, &offset) == (EXT_RECORDED_ALLOCATED >> 30)) { + else if (inode_bmap(dir, nf_pos >> (bits - 2), + &epos, &eloc, &elen, &offset) == + (EXT_RECORDED_ALLOCATED >> 30)) { block = udf_get_lb_pblock(dir->i_sb, eloc, offset); if ((++offset << bits) < elen) { if (UDF_I_ALLOCTYPE(dir) == ICBTAG_FLAG_AD_SHORT) epos.offset -= sizeof(short_ad); else if (UDF_I_ALLOCTYPE(dir) == ICBTAG_FLAG_AD_LONG) epos.offset -= sizeof(long_ad); - } else { + } else offset = 0; - } - if (!(fibh.sbh = fibh.ebh = udf_tread(dir->i_sb, block))) { + fibh.sbh = fibh.ebh = udf_tread(dir->i_sb, block); + if (!fibh.sbh) { brelse(epos.bh); return -EIO; } @@ -92,9 +94,11 @@ static int do_udf_readdir(struct inode * if (i + offset > (elen >> bits)) i = (elen >> bits) - offset; for (num = 0; i > 0; i--) { - block = udf_get_lb_pblock(dir->i_sb, eloc, offset + i); + block = udf_get_lb_pblock(dir->i_sb, eloc, + offset + i); tmp = udf_tgetblk(dir->i_sb, block); - if (tmp && !buffer_uptodate(tmp) && !buffer_locked(tmp)) + if (tmp && !buffer_uptodate(tmp) && + !buffer_locked(tmp)) bha[num++] = tmp; else brelse(tmp); @@ -126,16 +130,18 @@ static int do_udf_readdir(struct inode * liu = le16_to_cpu(cfi.lengthOfImpUse); lfi = cfi.lengthFileIdent; - if (fibh.sbh == fibh.ebh) { + if (fibh.sbh == fibh.ebh) nameptr = fi->fileIdent + liu; - } else { + else { int poffset; /* Unpaded ending offset */ - poffset = fibh.soffset + sizeof(struct fileIdentDesc) + liu + lfi; + poffset = fibh.soffset + sizeof(struct fileIdentDesc) + + liu + lfi; - if (poffset >= lfi) { - nameptr = (char *)(fibh.ebh->b_data + poffset - lfi); - } else { + if (poffset >= lfi) + nameptr = (char *)(fibh.ebh->b_data + + poffset - lfi); + else { nameptr = fname; memcpy(nameptr, fi->fileIdent + liu, lfi - poffset); @@ -168,12 +174,13 @@ static int do_udf_readdir(struct inode * } if (flen) { - if (filldir(dirent, fname, flen, filp->f_pos, iblock, dt_type) < 0) { + if (filldir(dirent, fname, flen, filp->f_pos, iblock, + dt_type) < 0) { if (fibh.sbh != fibh.ebh) brelse(fibh.ebh); brelse(fibh.sbh); brelse(epos.bh); - return 0; + return 0; } } } /* end while */ @@ -222,7 +229,8 @@ static int udf_readdir(struct file *filp lock_kernel(); if (filp->f_pos == 0) { - if (filldir(dirent, ".", 1, filp->f_pos, dir->i_ino, DT_DIR) < 0) { + if (filldir(dirent, ".", 1, filp->f_pos, dir->i_ino, + DT_DIR) < 0) { unlock_kernel(); return 0; } @@ -231,7 +239,7 @@ static int udf_readdir(struct file *filp result = do_udf_readdir(dir, filp, filldir, dirent); unlock_kernel(); - return result; + return result; } /* readdir and lookup functions */ _