bulk: Replace assert(0) -> g_assert_not_reached()

[PATCH 0/5] bulk: Replace assert(0) -> g_assert_not_reached()

Philippe Mathieu-Daudé posted 5 patches 1 year, 2 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230221232520.14480-1-philmd@linaro.org

Maintainers: Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Ani Sinha <ani@anisinha.ca>, Rob Herring <robh@kernel.org>, Peter Maydell <peter.maydell@linaro.org>, Michael Rolnik <mrolnik@gmail.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Yanan Wang <wangyanan55@huawei.com>, Richard Henderson <richard.henderson@linaro.org>, Helge Deller <deller@gmx.de>, Jason Wang <jasowang@redhat.com>, Fam Zheng <fam@euphon.net>, Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Markus Armbruster <armbru@redhat.com>, Huacai Chen <chenhuacai@kernel.org>, Aurelien Jarno <aurelien@aurel32.net>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, Daniel Henrique Barboza <danielhb413@gmail.com>, "Cédric Le Goater" <clg@kaod.org>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>, Corey Minyard <minyard@acm.org>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>

block/vvfat.c                       |   3 -
docs/spin/aio_notify_accept.promela |   6 +-
docs/spin/aio_notify_bug.promela    |   6 +-
hw/acpi/aml-build.c                 |   3 +-
hw/arm/highbank.c                   |   2 +-
hw/char/avr_usart.c                 |   2 +-
hw/core/numa.c                      |   2 +-
hw/net/i82596.c                     |   2 +-
hw/scsi/mptsas.c                    |   2 -
hw/virtio/virtio.c                  |   2 -
hw/watchdog/watchdog.c              |   2 +-
migration/migration-hmp-cmds.c      |   2 +-
migration/postcopy-ram.c            |  21 ++----
migration/ram.c                     |   8 +--
qobject/qlit.c                      |   2 +-
qobject/qnum.c                      |  12 ++--
scripts/checkpatch.pl               |   3 +
softmmu/rtc.c                       |   2 +-
target/mips/sysemu/physaddr.c       |   3 +-
target/mips/tcg/msa_helper.c        | 104 ++++++++++++++--------------
target/ppc/dfp_helper.c             |  12 ++--
target/ppc/mmu_helper.c             |   2 +-
tests/qtest/ipmi-bt-test.c          |   2 +-
tests/qtest/ipmi-kcs-test.c         |   4 +-
tests/qtest/rtl8139-test.c          |   2 +-
25 files changed, 96 insertions(+), 115 deletions(-)

Expand all Fold all

[PATCH 0/5] bulk: Replace assert(0) -> g_assert_not_reached()

Posted by Philippe Mathieu-Daudé 1 year, 2 months ago

Save contributors to post a patch each time clang
produce a -Werror=maybe-uninitialized warning on
assert(0). Replace by g_assert_not_reached()() and
prohibit '[g_]assert(0)'. Remove NDEBUG.

Philippe Mathieu-Daudé (4):
  scripts/checkpatch.pl: Do not allow assert(0)
  bulk: Replace [g_]assert(0) -> g_assert_not_reached()
  block/vvfat: Remove pointless check of NDEBUG
  hw: Remove mentions of NDEBUG

Pierrick Bouvier (1):
  target/ppc: fix warning with clang-15

 block/vvfat.c                       |   3 -
 docs/spin/aio_notify_accept.promela |   6 +-
 docs/spin/aio_notify_bug.promela    |   6 +-
 hw/acpi/aml-build.c                 |   3 +-
 hw/arm/highbank.c                   |   2 +-
 hw/char/avr_usart.c                 |   2 +-
 hw/core/numa.c                      |   2 +-
 hw/net/i82596.c                     |   2 +-
 hw/scsi/mptsas.c                    |   2 -
 hw/virtio/virtio.c                  |   2 -
 hw/watchdog/watchdog.c              |   2 +-
 migration/migration-hmp-cmds.c      |   2 +-
 migration/postcopy-ram.c            |  21 ++----
 migration/ram.c                     |   8 +--
 qobject/qlit.c                      |   2 +-
 qobject/qnum.c                      |  12 ++--
 scripts/checkpatch.pl               |   3 +
 softmmu/rtc.c                       |   2 +-
 target/mips/sysemu/physaddr.c       |   3 +-
 target/mips/tcg/msa_helper.c        | 104 ++++++++++++++--------------
 target/ppc/dfp_helper.c             |  12 ++--
 target/ppc/mmu_helper.c             |   2 +-
 tests/qtest/ipmi-bt-test.c          |   2 +-
 tests/qtest/ipmi-kcs-test.c         |   4 +-
 tests/qtest/rtl8139-test.c          |   2 +-
 25 files changed, 96 insertions(+), 115 deletions(-)

-- 
2.38.1