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. Thanks, Laurent _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel