[v3] cleanup: Tweak and re-run return_directly.cocci

[PATCH v3 0/3] cleanup: Tweak and re-run return_directly.cocci

Markus Armbruster posted 3 patches 1 year, 4 months ago

Diff against v1 v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221128094138.2824623-1-armbru@redhat.com

Maintainers: Fam Zheng <fam@euphon.net>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Alistair Francis <Alistair.Francis@wdc.com>, Bin Meng <bin.meng@windriver.com>, Palmer Dabbelt <palmer@dabbelt.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Yuval Shaia <yuval.shaia.ml@gmail.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Michael S. Tsirkin" <mst@redhat.com>, Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru>, "Alex Bennée" <alex.bennee@linaro.org>, Peter Xu <peterx@redhat.com>, David Hildenbrand <david@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Michael Rolnik <mrolnik@gmail.com>, Song Gao <gaosong@loongson.cn>, Xiaojuan Yang <yangxiaojuan@loongson.cn>, Aurelien Jarno <aurelien@aurel32.net>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Su Hang <suhang16@mails.ucas.ac.cn>, Zhang Chen <chen.zhang@intel.com>, Li Zhijian <lizhijian@fujitsu.com>, Stefan Hajnoczi <stefanha@redhat.com>

scripts/coccinelle/return_directly.cocci |  5 +--
include/hw/pci/pci.h                     |  7 +--
target/avr/cpu.h                         |  4 +-
block/vmdk.c                             | 28 +++++-------
hw/char/sifive_uart.c                    |  4 +-
hw/ppc/ppc4xx_sdram.c                    | 15 ++-----
hw/rdma/vmw/pvrdma_cmd.c                 | 57 +++++++++---------------
hw/virtio/vhost-user.c                   |  6 +--
migration/dirtyrate.c                    | 10 +----
migration/tls.c                          |  6 +--
replay/replay-time.c                     |  5 +--
semihosting/console.c                    |  4 +-
softmmu/memory.c                         | 11 +----
softmmu/physmem.c                        |  9 +---
target/loongarch/cpu.c                   |  4 +-
target/mips/tcg/dsp_helper.c             | 15 ++-----
target/riscv/debug.c                     |  6 +--
target/riscv/vector_helper.c             | 28 +++---------
tests/bench/benchmark-crypto-akcipher.c  |  6 +--
tests/qtest/erst-test.c                  |  5 +--
tests/qtest/hexloader-test.c             |  6 +--
tests/qtest/pvpanic-pci-test.c           |  6 +--
tests/qtest/pvpanic-test.c               |  6 +--
tests/qtest/test-filter-mirror.c         |  6 +--
tests/qtest/virtio-ccw-test.c            |  6 +--
tests/tcg/multiarch/sha512.c             |  9 +---
tools/virtiofsd/fuse_lowlevel.c          | 24 +++-------
27 files changed, 80 insertions(+), 218 deletions(-)

Expand all Fold all

[PATCH v3 0/3] cleanup: Tweak and re-run return_directly.cocci

Posted by Markus Armbruster 1 year, 4 months ago

v3:
* Rebased onto Greg's "[PATCH] 9pfs: Fix some return statements in the
  synth backend", which replaces my change to hw/9pfs/9p-synth.c in
  PATCH 1/3, along with a paragraph on checkpatch.pl in the commit
  message.

v2:
* Changes to imported files dropped [Philippe, Max]
* Handwritten change to vmdk_co_create() split off as PATCH 2/2 [Peter]
* New PATCH 3/2 [Zoltan]

Markus Armbruster (3):
  cleanup: Tweak and re-run return_directly.cocci
  block/vmdk: Simplify vmdk_co_create() to return directly
  ppc4xx_sdram: Simplify sdram_ddr_size() to return directly

 scripts/coccinelle/return_directly.cocci |  5 +--
 include/hw/pci/pci.h                     |  7 +--
 target/avr/cpu.h                         |  4 +-
 block/vmdk.c                             | 28 +++++-------
 hw/char/sifive_uart.c                    |  4 +-
 hw/ppc/ppc4xx_sdram.c                    | 15 ++-----
 hw/rdma/vmw/pvrdma_cmd.c                 | 57 +++++++++---------------
 hw/virtio/vhost-user.c                   |  6 +--
 migration/dirtyrate.c                    | 10 +----
 migration/tls.c                          |  6 +--
 replay/replay-time.c                     |  5 +--
 semihosting/console.c                    |  4 +-
 softmmu/memory.c                         | 11 +----
 softmmu/physmem.c                        |  9 +---
 target/loongarch/cpu.c                   |  4 +-
 target/mips/tcg/dsp_helper.c             | 15 ++-----
 target/riscv/debug.c                     |  6 +--
 target/riscv/vector_helper.c             | 28 +++---------
 tests/bench/benchmark-crypto-akcipher.c  |  6 +--
 tests/qtest/erst-test.c                  |  5 +--
 tests/qtest/hexloader-test.c             |  6 +--
 tests/qtest/pvpanic-pci-test.c           |  6 +--
 tests/qtest/pvpanic-test.c               |  6 +--
 tests/qtest/test-filter-mirror.c         |  6 +--
 tests/qtest/virtio-ccw-test.c            |  6 +--
 tests/tcg/multiarch/sha512.c             |  9 +---
 tools/virtiofsd/fuse_lowlevel.c          | 24 +++-------
 27 files changed, 80 insertions(+), 218 deletions(-)

-- 
2.37.3