[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 0/9] misc: Trivial static code analyzer fixes
From: |
Laurent Vivier |
Subject: |
Re: [PATCH v3 0/9] misc: Trivial static code analyzer fixes |
Date: |
Mon, 4 May 2020 12:04:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 |
Le 22/04/2020 à 15:31, Philippe Mathieu-Daudé a écrit :
> Fix trivial warnings reported by the Clang static code analyzer.
>
> Only patch #2 'blockdev: Remove dead assignment' misses review.
>
> The official Clang static code analyzer documentation is on:
> https://clang-analyzer.llvm.org/
>
> On Fedora I simply used it as:
>
> $ sudo dnf install clang-analyzer
> $ ../configure
> $ scan-build make
>
> Since v2:
> - Based on lvivier/trivial-patches-for-5.1
> - Removed dup patches from Kuhn Chenqun
> Since v1:
> - Addressed Markus/Zoltan/Aleksandar review comments
>
> Philippe Mathieu-Daudé (9):
> block: Avoid dead assignment
> blockdev: Remove dead assignment
> hw/i2c/pm_smbus: Remove dead assignment
> hw/input/adb-kbd: Remove dead assignment
> hw/ide/sii3112: Remove dead assignment
> hw/isa/i82378: Remove dead assignment
> hw/gpio/aspeed_gpio: Remove dead assignment
> hw/timer/stm32f2xx_timer: Remove dead assignment
> hw/timer/pxa2xx_timer: Add assertion to silent static analyzer warning
>
> block.c | 2 +-
> blockdev.c | 2 +-
> hw/gpio/aspeed_gpio.c | 2 +-
> hw/i2c/pm_smbus.c | 1 -
> hw/ide/sii3112.c | 5 +++--
> hw/input/adb-kbd.c | 6 +-----
> hw/isa/i82378.c | 8 ++++----
> hw/timer/pxa2xx_timer.c | 1 +
> hw/timer/stm32f2xx_timer.c | 1 -
> 9 files changed, 12 insertions(+), 16 deletions(-)
>
Applied to my trivial-patches branch.
Thanks,
Laurent