btrfs: remove pointless 'out' label from clone_finish_inode_update()
authorFilipe Manana <fdmanana@suse.com>
Fri, 16 May 2025 18:13:53 +0000 (19:13 +0100)
committerDavid Sterba <dsterba@suse.com>
Mon, 21 Jul 2025 13:31:06 +0000 (15:31 +0200)
The label is only used once and we can instead return directly where it's
used, besides the fact that all we do under the label is to return the
value of 'ret'. So get rid of the label and return directly.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.cz>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/reflink.c

index 62161beca5590359bf916386cbff5a298074a9ee..5eacd3584a8d321a1489922436d239ddce91978c 100644 (file)
@@ -46,11 +46,9 @@ static int clone_finish_inode_update(struct btrfs_trans_handle *trans,
        if (ret) {
                btrfs_abort_transaction(trans, ret);
                btrfs_end_transaction(trans);
-               goto out;
+               return ret;
        }
-       ret = btrfs_end_transaction(trans);
-out:
-       return ret;
+       return btrfs_end_transaction(trans);
 }
 
 static int copy_inline_to_page(struct btrfs_inode *inode,