[v2,00/10] trivial patchs for static code analyzer fixes

Message

Chen Qun Aug. 25, 2020, 11:24 a.m. UTC

Hi All,
   This series fix trivial warnings reported by the Clang static code analyzer.

Since v1:
- Patch1: Addressed Igor Mammedov review comment.
- Patch5：Addressed Raphael Norwitz review comment.
- Patch6: Addressed Philippe Mathieu-Daudé review comment.
- Patch7: Addressed Eric Auger review comment.
- Patch8: drop the patch "tcg/optimize:** " follow Richard Henderson comment.
- Patch9->Patch8: The patch code format is modified because spaces are missing.
- Patch11->Patch10: Addressed Gerd Hoffmann review comment.

Thanks.

Chen Qun (10):
  hw/arm/virt-acpi-build:Remove dead assignment in build_madt()
  hw/arm/omap1:Remove redundant statement in omap_clkdsp_read()
  target/arm/translate-a64:Remove dead assignment in
    handle_scalar_simd_shli()
  target/arm/translate-a64:Remove redundant statement in
    disas_simd_two_reg_misc_fp16()
  hw/virtio/vhost-user:Remove dead assignment in scrub_shadow_regions()
  hw/net/virtio-net:Remove redundant statement in
    virtio_net_rsc_tcp_ctrl_check()
  vfio/platform: Remove dead assignment in vfio_intp_interrupt()
  usb/bus: Remove dead assignment in usb_get_fw_dev_path()
  hw/intc: Remove redundant statement in exynos4210_combiner_read()
  hw/display/vga:Remove redundant statement in vga_draw_graphic()

 hw/arm/omap1.c                | 1 -
 hw/arm/virt-acpi-build.c      | 3 +--
 hw/display/vga.c              | 1 -
 hw/intc/exynos4210_combiner.c | 3 +--
 hw/net/virtio-net.c           | 1 -
 hw/usb/bus.c                  | 4 ++--
 hw/vfio/platform.c            | 2 +-
 hw/virtio/vhost-user.c        | 2 +-
 target/arm/translate-a64.c    | 7 ++-----
 9 files changed, 8 insertions(+), 16 deletions(-)