[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 20/22] sysemu/device_tree: Clean up local variable shadowi
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 20/22] sysemu/device_tree: Clean up local variable shadowing |
Date: |
Fri, 29 Sep 2023 06:58:50 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> Fix:
>
> hw/mips/boston.c:472:5: error: declaration shadows a local variable
> [-Werror,-Wshadow]
> qemu_fdt_setprop_cells(fdt, name, "reg", reg_base, reg_size);
> ^
> include/sysemu/device_tree.h:129:13: note: expanded from macro
> 'qemu_fdt_setprop_cells'
> int i;
> ^
> hw/mips/boston.c:461:9: note: previous declaration is here
> int i;
> ^
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> include/sysemu/device_tree.h | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/include/sysemu/device_tree.h b/include/sysemu/device_tree.h
> index ca5339beae..8eab395934 100644
> --- a/include/sysemu/device_tree.h
> +++ b/include/sysemu/device_tree.h
> @@ -126,10 +126,8 @@ int qemu_fdt_add_path(void *fdt, const char *path);
> #define qemu_fdt_setprop_cells(fdt, node_path, property, ...)
> \
> do {
> \
> uint32_t qdt_tmp[] = { __VA_ARGS__ };
> \
> - int i;
> \
> -
> \
> - for (i = 0; i < ARRAY_SIZE(qdt_tmp); i++) {
> \
> - qdt_tmp[i] = cpu_to_be32(qdt_tmp[i]);
> \
> + for (unsigned i_ = 0; i_ < ARRAY_SIZE(qdt_tmp); i_++) {
> \
> + qdt_tmp[i_] = cpu_to_be32(qdt_tmp[i_]);
> \
> }
> \
> qemu_fdt_setprop(fdt, node_path, property, qdt_tmp,
> \
> sizeof(qdt_tmp));
> \
You don't just rename the variable, but also adjust its type. When
we're dealing with at a huge changeset like the tree-wide -Wshadow=local
cleanup, I prefer to keep changes minimal to ease review as much as
possible. But it's sunk cost now, so
Reviewed-by: Markus Armbruster <armbru@redhat.com>
- Re: [PATCH v2 15/22] net/eth: Clean up local variable shadowing, (continued)
- [PATCH v2 16/22] crypto/cipher-gnutls.c: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- [PATCH v2 17/22] util/vhost-user-server: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- [PATCH v2 18/22] semihosting/arm-compat: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- [PATCH v2 19/22] linux-user/strace: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- [PATCH v2 20/22] sysemu/device_tree: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- Re: [PATCH v2 20/22] sysemu/device_tree: Clean up local variable shadowing,
Markus Armbruster <=
- [PATCH v2 21/22] softmmu/memory: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- [RFC PATCH v2 22/22] softmmu/physmem: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/04
- Re: [PATCH v2 00/22] (few more) Steps towards enabling -Wshadow, Markus Armbruster, 2023/09/29