|
| From: | Markus Armbruster |
| Subject: | Re: [PATCH v3] hw/i386: changes towards enabling -Wshadow=local for x86 machines |
| Date: | Wed, 04 Oct 2023 12:40:54 +0200 |
| User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Ani Sinha <anisinha@redhat.com> writes: > Code changes that addresses all compiler complaints coming from enabling > -Wshadow flags. Enabling -Wshadow catches cases of local variables shadowing > other local variables or parameters. These makes the code confusing and/or > adds > bugs that are difficult to catch. > > See also > > Subject: Help wanted for enabling -Wshadow=local > Message-Id: <87r0mqlf9x.fsf@pond.sub.org> > https://lore.kernel.org/qemu-devel/87r0mqlf9x.fsf@pond.sub.org > > CC: Markus Armbruster <armbru@redhat.com> > CC: Philippe Mathieu-Daude <philmd@linaro.org> > CC: mst@redhat.com > > Signed-off-by: Ani Sinha <anisinha@redhat.com> > Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> > Reviewed-by: Peter Xu <peterx@redhat.com> Queued. Thanks!
| [Prev in Thread] | Current Thread | [Next in Thread] |