On Tue, 2015-03-03 at 20:21 +0100, Fabian Frederick wrote:
> new_parent is only used in 0/1 context.
[]
> diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
[]
> @@ -2860,7 +2860,7 @@ xfs_rename(
>  {
>       xfs_trans_t     *tp = NULL;
>       xfs_mount_t     *mp = src_dp->i_mount;
> -     int             new_parent;             /* moving to a new dir */
> +     bool            new_parent;             /* moving to a new dir */
>       int             src_is_directory;       /* src_name is a directory */

Could convert src_is_directory to bool too.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to