Christoph Hellwig <hch@xxxxxxxxxxxxx> writes: > On Tue, Apr 11, 2023 at 08:51:24PM +0530, Ritesh Harjani wrote: >> >> + if ((flags & IOMAP_WRITE) && (offset + length > i_size_read(inode))) >> > >> > No need for the second set of inner braces here either. >> >> It's just avoids any confusion this way. > > Does it? Except for some really weird places it is very unusual in > the kernel. To me it really does add confusion. Ok, In that case, will change it. Thanks for the review! -ritesh
