Le 18/12/2019 à 13:01, Paolo Bonzini a écrit :
> On 18/12/19 12:57, Laurent Vivier wrote:
>> Le 09/12/2019 à 11:05, Thomas Huth a écrit :
>>> On 09/12/2019 10.49, Philippe Mathieu-Daudé wrote:
>>>> We don't enforce the -Wsign-conversion CPPFLAG, but it doesn't hurt
>>>> to avoid this warning:
>>>>
>>>> warning: implicit conversion changes signedness: 'int' to 'size_t' (aka
>>>> 'unsigned long') [-Wsign-conversion]
>>>>
>>>> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>>>> ---
>>>> hw/pci-host/i440fx.c | 5 ++---
>>>> 1 file changed, 2 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/hw/pci-host/i440fx.c b/hw/pci-host/i440fx.c
>>>> index fbdc563599..0cc80b276d 100644
>>>> --- a/hw/pci-host/i440fx.c
>>>> +++ b/hw/pci-host/i440fx.c
>>>> @@ -419,12 +419,11 @@ out:
>>>> static void igd_pt_i440fx_realize(PCIDevice *pci_dev, Error **errp)
>>>> {
>>>> uint32_t val = 0;
>>>> - int i, num;
>>>> + size_t i;
>>>> int pos, len;
>>>> Error *local_err = NULL;
>>>>
>>>> - num = ARRAY_SIZE(igd_host_bridge_infos);
>>>> - for (i = 0; i < num; i++) {
>>>> + for (i = 0; i < ARRAY_SIZE(igd_host_bridge_infos); i++) {
>>>> pos = igd_host_bridge_infos[i].offset;
>>>> len = igd_host_bridge_infos[i].len;
>>>> host_pci_config_read(pos, len, &val, &local_err);
>>>>
>>>
>>> Reviewed-by: Thomas Huth <th...@redhat.com>
>>>
>>>
>>
>> Applied to my trivial-patches branch.
>
> No need to, I've already queued the whole series.
>
ok, I've removed them from my queue.
Thanks,
Laurent
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
