diff options
| author | Markus Armbruster <armbru@redhat.com> | 2023-09-21 14:13:11 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2023-09-29 08:13:57 +0200 |
| commit | fb2575f95411644abe7f0606594035b63a5132ad (patch) | |
| tree | e9e0fe4dc77d0d3575436232a7ce4d8d35a20c82 /block/stream.c | |
| parent | d25b99c72b0178ce0e0c766b07011102dbbacf6a (diff) | |
block: Clean up local variable shadowing
Local variables shadowing other local variables or parameters make the
code needlessly hard to understand. Tracked down with -Wshadow=local.
Clean up: delete inner declarations when they are actually redundant,
else rename variables.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Acked-by: Anthony PERARD <anthony.perard@citrix.com>
Acked-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Message-ID: <20230921121312.1301864-7-armbru@redhat.com>
Diffstat (limited to 'block/stream.c')
| -rw-r--r-- | block/stream.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/block/stream.c b/block/stream.c index e4da214f1f..133cb72fb4 100644 --- a/block/stream.c +++ b/block/stream.c @@ -292,7 +292,6 @@ void stream_start(const char *job_id, BlockDriverState *bs, /* Make sure that the image is opened in read-write mode */ bs_read_only = bdrv_is_read_only(bs); if (bs_read_only) { - int ret; /* Hold the chain during reopen */ if (bdrv_freeze_backing_chain(bs, above_base, errp) < 0) { return; |