diff options
| author |   Filipe Manana <fdmanana@suse.com> | 2025-02-18 15:49:11 +0000 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2025-03-18 20:35:46 +0100 |
| commit | de6d3a5b78757a462ebcebdcbb52ffce7dd61c70 (patch) | |
| tree | b2c775cd7539df211013e17ccdf103d9271f72ca /fs/btrfs/send.c | |
| parent | btrfs: send: simplify return logic from record_deleted_ref_if_needed() (diff) | |
| download | wireguard-linux-de6d3a5b78757a462ebcebdcbb52ffce7dd61c70.tar.xz wireguard-linux-de6d3a5b78757a462ebcebdcbb52ffce7dd61c70.zip | |
btrfs: send: simplify return logic from record_new_ref()
There is no need to have an 'out' label and jump into it since there are
no resource cleanups to perform (release locks, free memory, etc), so
make this simpler by removing the label and goto and instead return
directly.
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/send.c')
| -rw-r--r-- | fs/btrfs/send.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index b715557ec720..181a234e3a5e 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -4746,11 +4746,9 @@ static int record_new_ref(struct send_ctx *sctx) ret = iterate_inode_ref(sctx->send_root, sctx->left_path, sctx->cmp_key, 0, record_new_ref_if_needed, sctx); if (ret < 0) - goto out; - ret = 0; + return ret; -out: - return ret; + return 0; } static int record_deleted_ref(struct send_ctx *sctx) |