bcachefs: remove redundant initializations of variables start_offset and end_offset
authorColin Ian King <colin.i.king@gmail.com>
Tue, 12 Sep 2023 12:37:42 +0000 (13:37 +0100)
committerKent Overstreet <kent.overstreet@linux.dev>
Sun, 22 Oct 2023 21:10:13 +0000 (17:10 -0400)
The variables start_offset and end_offset are being initialized with
values that are never read, they being re-assigned later on. The
initializations are redundant and can be removed.

Cleans up clang-scan build warnings:
fs/bcachefs/fs-io.c:243:11: warning: Value stored to 'start_offset' during
its initialization is never read [deadcode.DeadStores]
fs/bcachefs/fs-io.c:244:11: warning: Value stored to 'end_offset' during
its initialization is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
fs/bcachefs/fs-io.c

index b36513e..ffe9206 100644 (file)
@@ -241,8 +241,8 @@ static int __bch2_truncate_folio(struct bch_inode_info *inode,
        struct bch_fs *c = inode->v.i_sb->s_fs_info;
        struct address_space *mapping = inode->v.i_mapping;
        struct bch_folio *s;
-       unsigned start_offset = start & (PAGE_SIZE - 1);
-       unsigned end_offset = ((end - 1) & (PAGE_SIZE - 1)) + 1;
+       unsigned start_offset;
+       unsigned end_offset;
        unsigned i;
        struct folio *folio;
        s64 i_sectors_delta = 0;