]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
btrfs: send: only use boolean variables at process_recorded_refs()
authorFilipe Manana <fdmanana@suse.com>
Wed, 12 Feb 2025 11:31:59 +0000 (11:31 +0000)
committerDavid Sterba <dsterba@suse.com>
Tue, 18 Mar 2025 19:35:45 +0000 (20:35 +0100)
We have several local variables at process_recorded_refs() that are used
as booleans, with some of them having a 'bool' type while two of them
having an 'int' type. Change this to make them all use the 'bool' type
which is more clear and to make everything more consistent.

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

index 3aa2877f8c80fe04227bbe9b1959187f1679f7ea..6e27a7d77b25db9160cea3b456940371aad717ce 100644 (file)
@@ -4147,9 +4147,9 @@ static int process_recorded_refs(struct send_ctx *sctx, int *pending_move)
        u64 ow_inode = 0;
        u64 ow_gen;
        u64 ow_mode;
-       int did_overwrite = 0;
-       int is_orphan = 0;
        u64 last_dir_ino_rm = 0;
+       bool did_overwrite = false;
+       bool is_orphan = false;
        bool can_rename = true;
        bool orphanized_dir = false;
        bool orphanized_ancestor = false;
@@ -4191,14 +4191,14 @@ static int process_recorded_refs(struct send_ctx *sctx, int *pending_move)
                if (ret < 0)
                        goto out;
                if (ret)
-                       did_overwrite = 1;
+                       did_overwrite = true;
        }
        if (sctx->cur_inode_new || did_overwrite) {
                ret = gen_unique_name(sctx, sctx->cur_ino,
                                sctx->cur_inode_gen, valid_path);
                if (ret < 0)
                        goto out;
-               is_orphan = 1;
+               is_orphan = true;
        } else {
                ret = get_cur_path(sctx, sctx->cur_ino, sctx->cur_inode_gen,
                                valid_path);
@@ -4421,7 +4421,7 @@ static int process_recorded_refs(struct send_ctx *sctx, int *pending_move)
                        ret = send_rename(sctx, valid_path, cur->full_path);
                        if (ret < 0)
                                goto out;
-                       is_orphan = 0;
+                       is_orphan = false;
                        ret = fs_path_copy(valid_path, cur->full_path);
                        if (ret < 0)
                                goto out;
@@ -4482,7 +4482,7 @@ static int process_recorded_refs(struct send_ctx *sctx, int *pending_move)
                                        sctx->cur_inode_gen, valid_path);
                        if (ret < 0)
                                goto out;
-                       is_orphan = 1;
+                       is_orphan = true;
                }
 
                list_for_each_entry(cur, &sctx->deleted_refs, list) {