Commit c01b7114 authored by Dan Carpenter's avatar Dan Carpenter Committed by David Sterba
Browse files

btrfs: return a literal instead of a variable in btrfs_init_dev_replace() 



This is just a small clean up, it doesn't change how the code works.
Originally this code had a goto so we needed to set "ret = 0;" but now
it returns directly and so we can simplify it a bit by doing a
"return 0;" and removing the assignment.

Reviewed-by: default avatarAnand Jain <anand.jain@oracle.com>
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: default avatarDavid Sterba <dsterba@suse.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent 477a7a9c

fs/btrfs/dev-replace.c

+1 −2
Original line number Diff line number Diff line
@@ -103,7 +103,6 @@ int btrfs_init_dev_replace(struct btrfs_fs_info *fs_info) 
			"found replace target device without a valid replace item");

			return -EUCLEAN;

		}

		ret = 0;

		dev_replace->replace_state =

			BTRFS_IOCTL_DEV_REPLACE_STATE_NEVER_STARTED;

		dev_replace->cont_reading_from_srcdev_mode =
@@ -120,7 +119,7 @@ int btrfs_init_dev_replace(struct btrfs_fs_info *fs_info) 
		dev_replace->tgtdev = NULL;

		dev_replace->is_valid = 0;

		dev_replace->item_needs_writeback = 0;

		return ret;

		return 0;

	}

	slot = path->slots[0];

	eb = path->nodes[0];