[PATCH] Make out of range type conversions explicit
Pedro Alves
pedro@palves.net
Fri Jul 3 14:05:33 GMT 2020
On 7/3/20 2:09 PM, Gary Benson wrote:
> Pedro Alves wrote:
>> On 7/3/20 10:30 AM, Gary Benson wrote:
>>> Pedro Alves wrote:
>>>> diff --git a/gdb/testsuite/gdb.base/structs2.c b/gdb/testsuite/gdb.base/structs2.c
>>>> index 7c8be035221..aac7bce8c15 100644
>>>> --- a/gdb/testsuite/gdb.base/structs2.c
>>>> +++ b/gdb/testsuite/gdb.base/structs2.c
>>>> @@ -13,7 +13,7 @@ main ()
>>>>
>>>> bkpt = 0;
>>>> param_reg (120, 130, 32000, 33000);
>>>> - param_reg (130, 120, 33000, 32000);
>>>> + param_reg (-120, 130, -32000, 33000);
>>>>
>>>> return 0;
>>>> }
>>>
>>> On first glance I thought this was a copy-paste error. I don't
>>> think it is, but, just in case, can you confirm the change above
>>> is what you intended?
>>
>> What do you mean by "this"? If you mean, the proposed change
>> instead of:
>>
>> param_reg (120, 130, 32000, 33000);
>> - param_reg (130, 120, 33000, 32000);
>> + param_reg (-130, 120, -33000, 32000);
>>
>> It's just that -130 and -33000 overflows, so I swapped the
>> numbers back.
>
> Cool, thank you.
>
I've merged both patches.
Thanks,
Pedro Alves
More information about the Gdb-patches
mailing list