[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qom: Fix "typddef" typo
From: |
Laurent Vivier |
Subject: |
Re: [PATCH] qom: Fix "typddef" typo |
Date: |
Mon, 12 Oct 2020 17:24:05 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
Le 12/10/2020 à 16:45, Daniel P. Berrangé a écrit :
> On Wed, Oct 07, 2020 at 06:02:55PM -0400, Eduardo Habkost wrote:
>> Fix typo introduced in the C11 #ifdef for qemu_max_align_t. It
>> never caused any problems because we always compile using
>> -std=gnu99.
>>
>> Fixes: 4c880f363e9e ("qom: Allow objects to be allocated with increased
>> alignment")
>> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
>> ---
>> qom/object.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qom/object.c b/qom/object.c
>> index c335dce7e4..125dabd28b 100644
>> --- a/qom/object.c
>> +++ b/qom/object.c
>> @@ -692,7 +692,7 @@ static void object_finalize(void *data)
>>
>> /* Find the minimum alignment guaranteed by the system malloc. */
>> #if __STDC_VERSION__ >= 201112L
>> -typddef max_align_t qemu_max_align_t;
>> +typedef max_align_t qemu_max_align_t;
>> #else
>
> Why do we need / have this first part of the #if at all ? We
> unconditionally add -std=gnu99, so surely this has always
> been dead code and can just be better deleted.
>
I've removed the patch from my queue.
But perhaps it will be needed in the future if we switch to a newer
standard version?
Thanks,
Laurent