Le 02/03/2020 à 14:07, Chen Qun a écrit : > Clang static code analyzer show warning: > hw/scsi/esp-pci.c:198:9: warning: Value stored to 'size' is never read > size = 4; > ^ ~ > > Reported-by: Euler Robot <euler.ro...@huawei.com> > Signed-off-by: Chen Qun <kuhn.chen...@huawei.com> > --- > Cc: Paolo Bonzini <pbonz...@redhat.com> > Cc:Fam Zheng <f...@euphon.net> > --- > hw/scsi/esp-pci.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/hw/scsi/esp-pci.c b/hw/scsi/esp-pci.c > index d5a1f9e017..2e6cc07d4e 100644 > --- a/hw/scsi/esp-pci.c > +++ b/hw/scsi/esp-pci.c > @@ -195,7 +195,6 @@ static void esp_pci_io_write(void *opaque, hwaddr addr, > val <<= shift; > val |= current & ~(mask << shift); > addr &= ~3; > - size = 4; > }
perhaps a "g_assert(size >= 4)" instead would be cleaner to mute the warning? I think it's a good point to update the size if in the future the code below is modified to use size. Thanks, Laurent
