Skip to content

Commit

Permalink
qemu-img: remove unused variable
Browse files Browse the repository at this point in the history
Variable block_count used in img_dd function is only incremented but never read.
This causes 'Unused but set variable' warning on Clang 15.0.1 compiler.

Removing the variable to prevent the warning.

Signed-off-by: Miroslav Rezanina <[email protected]>
Reviewed-by: Thomas Huth <[email protected]>
Message-Id: <e86d5b57f9d13bde995c616a533b876f1fb8a527.1668009030.git.mrezanin@redhat.com>
Signed-off-by: Thomas Huth <[email protected]>
  • Loading branch information
LaneWolf authored and huth committed Nov 11, 2022
1 parent 6083dca commit 0f48c47
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions qemu-img.c
Original file line number Diff line number Diff line change
Expand Up @@ -4922,7 +4922,7 @@ static int img_dd(int argc, char **argv)
const char *out_fmt = "raw";
const char *fmt = NULL;
int64_t size = 0;
int64_t block_count = 0, out_pos, in_pos;
int64_t out_pos, in_pos;
bool force_share = false;
struct DdInfo dd = {
.flags = 0,
Expand Down Expand Up @@ -5122,7 +5122,7 @@ static int img_dd(int argc, char **argv)

in.buf = g_new(uint8_t, in.bsz);

for (out_pos = 0; in_pos < size; block_count++) {
for (out_pos = 0; in_pos < size; ) {
int bytes = (in_pos + in.bsz > size) ? size - in_pos : in.bsz;

ret = blk_pread(blk1, in_pos, bytes, in.buf, 0);
Expand Down

0 comments on commit 0f48c47

Please sign in to comment.