On 12/13 2013 19:14 PM, Christoph Hellwig wrote: >> + * Loop over the new block(s), filling in the inodes. For small block >> + * sizes, manipulate the inodes in buffers which are multiples of the >> + * blocks size. >> */ >> + blks_per_cluster = xfs_icluster_size_fsb(mp); >> + inodes_per_cluster = blks_per_cluster << mp->m_sb.sb_inopblog; >> + nbufs = length / blks_per_cluster; > > I don't think there's a point to have the nbufs variable as it's used > just once as the loop termination condition. Thanks for the review, and that sounds make sense to me given that compiler would mostly likely optimize this situation to calculate nbufs only once in the loop, but I also consider that might affect a bit readabilities in either case... Thanks, -Jeff _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
