On Wed, Oct 04, 2023 at 02:00:14PM +0200, Philippe Mathieu-Daudé wrote: > Fix: > > util/cutils.c:1147:17: error: declaration shadows a variable in the global > scope [-Werror,-Wshadow] > const char *exec_dir = qemu_get_exec_dir(); > ^ > util/cutils.c:1035:20: note: previous declaration is here > static const char *exec_dir; > ^ > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > util/cutils.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/util/cutils.c b/util/cutils.c > index 25373198ad..b44718a6a2 100644 > --- a/util/cutils.c > +++ b/util/cutils.c > @@ -1144,7 +1144,6 @@ char *get_relocated_path(const char *dir) > { > size_t prefix_len = strlen(CONFIG_PREFIX); > const char *bindir = CONFIG_BINDIR; > - const char *exec_dir = qemu_get_exec_dir(); > GString *result; > int len_dir, len_bindir; > > --
Took me a few seconds to see it, but since we have this just a few lines before: const char *qemu_get_exec_dir(void) { return exec_dir; } the deletion of the redundant local variable is just fine. Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. Virtualization: qemu.org | libguestfs.org