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

Chen Qun posted 10 patches 3 years, 8 months ago
Test docker-quick@centos7 failed
Test docker-mingw@fedora failed
Test checkpatch failed
Test FreeBSD failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200825112447.126308-1-kuhn.chenqun@huawei.com
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>, Igor Mitsyanko <i.mitsyanko@gmail.com>, Igor Mammedov <imammedo@redhat.com>, Shannon Zhao <shannon.zhaosl@gmail.com>, Alex Williamson <alex.williamson@redhat.com>, Peter Maydell <peter.maydell@linaro.org>
There is a newer version of this series
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(-)
[PATCH v2 00/10] trivial patchs for static code analyzer fixes
Posted by Chen Qun 3 years, 8 months ago
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(-)

-- 
2.23.0