[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local
From: |
Ani Sinha |
Subject: |
Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local |
Date: |
Fri, 29 Sep 2023 11:24:19 +0530 |
> On 29-Sep-2023, at 11:17 AM, Markus Armbruster <armbru@redhat.com> wrote:
>
> Ani Sinha <anisinha@redhat.com> writes:
>
>> Code changes in acpi 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.
>>
>> The code is tested to build with and without the flag turned on.
>>
>> CC: Markus Armbruster <armbru@redhat.com>
>> CC: Philippe Mathieu-Daude <philmd@linaro.org>
>> CC: mst@redhat.com
>> CC: imammedo@redhat.com
>> Message-Id: <87r0mqlf9x.fsf@pond.sub.org>
>
> This is my "Help wanted for enabling -Wshadow=local" post.
Yes indeed. I wanted to refer to that thread for context in the commit log.
>
> A commit's Message-Id tag is supposed to point to the patch submission
> e-mail, and git-am will add that:
>
> Message-ID: <20230922124203.127110-1-anisinha@redhat.com>
>
> We'll have two Message-IDs then. Confusing.
>
> Could perhaps use
>
> See-also: Message-Id: <87r0mqlf9x.fsf@pond.sub.org>
>
> but I doubt it's worth the bother.
OK
>
>> Signed-off-by: Ani Sinha <anisinha@redhat.com>
>
> Queued less the extra Message-Id, thanks!
- [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Ani Sinha, 2023/09/22
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Michael S. Tsirkin, 2023/09/22
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Markus Armbruster, 2023/09/29
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local,
Ani Sinha <=
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Markus Armbruster, 2023/09/29
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Ani Sinha, 2023/09/29
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Markus Armbruster, 2023/09/29
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Ani Sinha, 2023/09/29
- Re: [PATCH] hw/acpi: changes towards enabling -Wshadow=local, Markus Armbruster, 2023/09/29