On Thu, Dec 01, 2016 at 11:02:19PM +0100, Richard Weinberger wrote: > > + if (new_dir != old_dir) { > + fst_inode_ui->parent_inum = new_dir->i_ino; > + snd_inode_ui->parent_inum = old_dir->i_ino; > + } > + > if (old_dir != new_dir) { > if (S_ISDIR(fst_inode->i_mode) && !S_ISDIR(snd_inode->i_mode)) { > inc_nlink(new_dir); Insignificant thing, but use the existing if-statment instead? Cheers, Marcus Folkesson -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html