Re: [PATCH 0/4] aspeed: Clean up local variable shadowing
Cédric Le Goater writes: > Hello, > > Here are cleanups for local variable shadowing warnings in aspeed models. > > Joel, Andrew, > > Could you please double check patch 4 ? > > Thanks, > > C. Queued, thanks!
Re: [PATCH 0/4] aspeed: Clean up local variable shadowing
Cédric Le Goater writes: > On 9/23/23 09:13, Markus Armbruster wrote: >> Cédric Le Goater writes: >> >>> On 9/22/23 20:20, Philippe Mathieu-Daudé wrote: On 22/9/23 17:59, Cédric Le Goater wrote: > Hello, > > Here are cleanups for local variable shadowing warnings in aspeed models. > > Joel, Andrew, > > Could you please double check patch 4 ? Could Markus' MAKE_IDENTFIER() help there? >>> >>> ah ! you typed too fast and I also read too fast, as : >>> >>>MARKUS_IDENTIFIER() >>> >>> and I liked it :) >> >> LOL >> >>>but what is MAKE_IDENTIFIER ? really, please explain. >> >> Philippe is referring to >> >> [PATCH v3 7/7] qobject atomics osdep: Make a few macros more hygienic >> Message-ID: <20230921121312.1301864-8-arm...@redhat.com> >> >> which tweaks MAX() to permit nesting without shadowing. Your PATCH 4 >> may not be needed if you base on it. > > We don't need the nested MAX(). PATCH 4 is a cleanup which happens > to remove the shadowing. Let's keep it the way it is. > >> MAKE_IDENTIFIER() is a helper macro introduced in that patch. >> You can fetch the patch from https://repo.or.cz/qemu/armbru.git branch >> shadow-next, along with collected other shadowing patches. > > OK. > > Would you prefer maintainers to include the shadowing changes in a > potential PR they would send or would you rather take care of it in > a PR of your own ? I'm happy to collect patches and do pull requests. I don't mind maintainers merging patches for their subsystems; interference should be minimal. Thanks!
Re: [PATCH 0/4] aspeed: Clean up local variable shadowing
On 9/23/23 09:13, Markus Armbruster wrote: Cédric Le Goater writes: On 9/22/23 20:20, Philippe Mathieu-Daudé wrote: On 22/9/23 17:59, Cédric Le Goater wrote: Hello, Here are cleanups for local variable shadowing warnings in aspeed models. Joel, Andrew, Could you please double check patch 4 ? Could Markus' MAKE_IDENTFIER() help there? ah ! you typed too fast and I also read too fast, as : MARKUS_IDENTIFIER() and I liked it :) LOL but what is MAKE_IDENTIFIER ? really, please explain. Philippe is referring to [PATCH v3 7/7] qobject atomics osdep: Make a few macros more hygienic Message-ID: <20230921121312.1301864-8-arm...@redhat.com> which tweaks MAX() to permit nesting without shadowing. Your PATCH 4 may not be needed if you base on it. We don't need the nested MAX(). PATCH 4 is a cleanup which happens to remove the shadowing. Let's keep it the way it is. MAKE_IDENTIFIER() is a helper macro introduced in that patch. You can fetch the patch from https://repo.or.cz/qemu/armbru.git branch shadow-next, along with collected other shadowing patches. OK. Would you prefer maintainers to include the shadowing changes in a potential PR they would send or would you rather take care of it in a PR of your own ? Thanks, C.
Re: [PATCH 0/4] aspeed: Clean up local variable shadowing
Cédric Le Goater writes: > On 9/22/23 20:20, Philippe Mathieu-Daudé wrote: >> On 22/9/23 17:59, Cédric Le Goater wrote: >>> Hello, >>> >>> Here are cleanups for local variable shadowing warnings in aspeed models. >>> >>> Joel, Andrew, >>> >>> Could you please double check patch 4 ? >> Could Markus' MAKE_IDENTFIER() help there? > > ah ! you typed too fast and I also read too fast, as : > > MARKUS_IDENTIFIER() > > and I liked it :) LOL > but what is MAKE_IDENTIFIER ? really, please explain. Philippe is referring to [PATCH v3 7/7] qobject atomics osdep: Make a few macros more hygienic Message-ID: <20230921121312.1301864-8-arm...@redhat.com> which tweaks MAX() to permit nesting without shadowing. Your PATCH 4 may not be needed if you base on it. MAKE_IDENTIFIER() is a helper macro introduced in that patch. You can fetch the patch from https://repo.or.cz/qemu/armbru.git branch shadow-next, along with collected other shadowing patches. Questions?
Re: [PATCH 0/4] aspeed: Clean up local variable shadowing
On 9/22/23 20:20, Philippe Mathieu-Daudé wrote: On 22/9/23 17:59, Cédric Le Goater wrote: Hello, Here are cleanups for local variable shadowing warnings in aspeed models. Joel, Andrew, Could you please double check patch 4 ? Could Markus' MAKE_IDENTFIER() help there? ah ! you typed too fast and I also read too fast, as : MARKUS_IDENTIFIER() and I liked it :) but what is MAKE_IDENTIFIER ? really, please explain. Thanks, C.
Re: [PATCH 0/4] aspeed: Clean up local variable shadowing
On 22/9/23 17:59, Cédric Le Goater wrote: Hello, Here are cleanups for local variable shadowing warnings in aspeed models. Joel, Andrew, Could you please double check patch 4 ? Could Markus' MAKE_IDENTFIER() help there? Thanks, C. Cédric Le Goater (4): aspeed/i2c: Clean up local variable shadowing aspeed: Clean up local variable shadowing aspeed/i3c: Rename variable shadowing a local aspeed/timer: Clean up local variable shadowing hw/arm/aspeed_ast2600.c | 10 +- hw/i2c/aspeed_i2c.c | 1 - hw/misc/aspeed_i3c.c| 6 +++--- hw/timer/aspeed_timer.c | 2 +- 4 files changed, 9 insertions(+), 10 deletions(-)