There is no point in having the label since all it does is return the
value in the 'ret' variable. Instead make every goto return directly
and remove the label.
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
map = btrfs_find_chunk_map(fs_info, 0, U64_MAX);
/* No chunk at all? Return false anyway */
- if (!map) {
- ret = false;
- goto out;
- }
+ if (!map)
+ return false;
+
while (map) {
int missing = 0;
int max_tolerated;
"chunk %llu missing %d devices, max tolerance is %d for writable mount",
map->start, missing, max_tolerated);
btrfs_free_chunk_map(map);
- ret = false;
- goto out;
+ return false;
}
next_start = map->start + map->chunk_len;
btrfs_free_chunk_map(map);
map = btrfs_find_chunk_map(fs_info, next_start, U64_MAX - next_start);
}
-out:
+
return ret;
}