* [e2fsprogs PATCH] resize2fs: remove unused variable from adjust_superblock()
@ 2023-01-04 9:03 Eric Biggers
2023-01-04 10:35 ` Lukas Czerner
0 siblings, 1 reply; 2+ messages in thread
From: Eric Biggers @ 2023-01-04 9:03 UTC (permalink / raw
To: linux-ext4
From: Eric Biggers <ebiggers@google.com>
In adjust_superblock(), the 'group_block' variable is declared and set,
but it is never actually used. Remove it.
This addresses the following compiler warning with clang -Wall:
resize2fs.c:1119:11: warning: variable 'group_block' set but not used [-Wunused-but-set-variable]
blk64_t group_block;
^
Signed-off-by: Eric Biggers <ebiggers@google.com>
---
resize/resize2fs.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/resize/resize2fs.c b/resize/resize2fs.c
index 243cd777..5eeb7d44 100644
--- a/resize/resize2fs.c
+++ b/resize/resize2fs.c
@@ -1116,7 +1116,6 @@ static errcode_t adjust_superblock(ext2_resize_t rfs, blk64_t new_size)
ext2_filsys fs = rfs->new_fs;
int adj = 0;
errcode_t retval;
- blk64_t group_block;
unsigned long i;
unsigned long max_group;
@@ -1181,8 +1180,6 @@ static errcode_t adjust_superblock(ext2_resize_t rfs, blk64_t new_size)
goto errout;
memset(rfs->itable_buf, 0, fs->blocksize * fs->inode_blocks_per_group);
- group_block = ext2fs_group_first_block2(fs,
- rfs->old_fs->group_desc_count);
adj = rfs->old_fs->group_desc_count;
max_group = fs->group_desc_count - adj;
if (rfs->progress) {
@@ -1209,7 +1206,6 @@ static errcode_t adjust_superblock(ext2_resize_t rfs, blk64_t new_size)
if (retval)
goto errout;
}
- group_block += fs->super->s_blocks_per_group;
}
io_channel_flush(fs->io);
retval = 0;
--
2.39.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [e2fsprogs PATCH] resize2fs: remove unused variable from adjust_superblock()
2023-01-04 9:03 [e2fsprogs PATCH] resize2fs: remove unused variable from adjust_superblock() Eric Biggers
@ 2023-01-04 10:35 ` Lukas Czerner
0 siblings, 0 replies; 2+ messages in thread
From: Lukas Czerner @ 2023-01-04 10:35 UTC (permalink / raw
To: Eric Biggers; +Cc: linux-ext4
On Wed, Jan 04, 2023 at 01:03:51AM -0800, Eric Biggers wrote:
> From: Eric Biggers <ebiggers@google.com>
>
> In adjust_superblock(), the 'group_block' variable is declared and set,
> but it is never actually used. Remove it.
>
> This addresses the following compiler warning with clang -Wall:
>
> resize2fs.c:1119:11: warning: variable 'group_block' set but not used [-Wunused-but-set-variable]
> blk64_t group_block;
Looks good thanks.
Reviewed-by: Lukas Czerner <lczerner@redhat.com>
> ^
> Signed-off-by: Eric Biggers <ebiggers@google.com>
> ---
> resize/resize2fs.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/resize/resize2fs.c b/resize/resize2fs.c
> index 243cd777..5eeb7d44 100644
> --- a/resize/resize2fs.c
> +++ b/resize/resize2fs.c
> @@ -1116,7 +1116,6 @@ static errcode_t adjust_superblock(ext2_resize_t rfs, blk64_t new_size)
> ext2_filsys fs = rfs->new_fs;
> int adj = 0;
> errcode_t retval;
> - blk64_t group_block;
> unsigned long i;
> unsigned long max_group;
>
> @@ -1181,8 +1180,6 @@ static errcode_t adjust_superblock(ext2_resize_t rfs, blk64_t new_size)
> goto errout;
>
> memset(rfs->itable_buf, 0, fs->blocksize * fs->inode_blocks_per_group);
> - group_block = ext2fs_group_first_block2(fs,
> - rfs->old_fs->group_desc_count);
> adj = rfs->old_fs->group_desc_count;
> max_group = fs->group_desc_count - adj;
> if (rfs->progress) {
> @@ -1209,7 +1206,6 @@ static errcode_t adjust_superblock(ext2_resize_t rfs, blk64_t new_size)
> if (retval)
> goto errout;
> }
> - group_block += fs->super->s_blocks_per_group;
> }
> io_channel_flush(fs->io);
> retval = 0;
> --
> 2.39.0
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-01-04 10:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-04 9:03 [e2fsprogs PATCH] resize2fs: remove unused variable from adjust_superblock() Eric Biggers
2023-01-04 10:35 ` Lukas Czerner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).