qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 00/44] Less clumsy error checking
@ 2020-07-02 15:49 Markus Armbruster
  2020-07-02 15:49 ` [PATCH v2 01/44] error: Improve examples in error.h's big comment Markus Armbruster
                   ` (46 more replies)
  0 siblings, 47 replies; 88+ messages in thread
From: Markus Armbruster @ 2020-07-02 15:49 UTC (permalink / raw)
  To: qemu-devel
  Cc: peter.maydell, vsementsov, berrange, ehabkost, qemu-block,
	pbonzini

When the Error API was created, we adopted the (unwritten) rule to
return void when the function returns no useful value on success,
unlike GError, which recommends to return true on success and false on
error then.

When a function returns a distinct error value, say false, a checked
call that passes the error up looks like

    if (!frobnicate(..., errp)) {
        handle the error...
    }

When it returns void, we need

    Error *err = NULL;

    frobnicate(..., &err);
    if (err) {
        handle the error...
        error_propagate(errp, err);
    }

Not only is this more verbose, it also creates an Error object even
when @errp is null, &error_abort or &error_fatal.

People got tired of the additional boilerplate, and started to ignore
the unwritten rule.  The result is confusion among developers about
the preferred usage.

This series adopts the GError rule (in writing), and updates a
substantial amount of code to honor the rule.  Cuts the number of
error_propagate() calls nearly by half.  The diffstat speaks for
itself.

Based on my "[PULL 00/28] Error reporting patches patches for
2020-07-02".  Also available from my public repository
https://repo.or.cz/qemu/armbru.git on branch error-smooth.

Based-on: Message-Id: <20200702110931.2953148-1-armbru@redhat.com>

v2:
* Rebased
* Coccinelle scripts reworked, patches sliced and diced for
  reviewability:
  Old PATCH 03+17+23-24+35+38-39+42 split into "Use returned bool to
  check for failure" parts [PATCH 03+13+17-18+25+28-29+42], and
  "Eliminate error_propagate()" parts.  The latter combined with old
  PATCH 06-07+18-19+29+43 are now [PATCH 33-37].  The end result is
  almost identical.  Some R-bys dropped; sorry!
* Drop variables as they become unused [Vladimir]
* PATCH 01: Comment typos fixed [Greg]
* PATCH 09: Simplify further by cutting out variables [Eric]
* PATCH 11: opt_set() renamed to opt_validate(), redundant parameter
  dropped [Vladimir], R-by kept
* PATCH 16: Comment typos fixed, indentation tidied up [Eric]
* PATCH 23: Assertion dropped [Eric], incorrect hunk backed out
* PATCH 26: Regenerated after rebase; note additional Coccinelle
  hiccup in the commit message
* PATCH 27: Indentation tiedied up [Eric]
* Left for later: followup to fix nearby typos and such [Eric]

Markus Armbruster (44):
  error: Improve examples in error.h's big comment
  error: Document Error API usage rules
  qdev: Use returned bool to check for qdev_realize() etc. failure
  macio: Tidy up error handling in macio_newworld_realize()
  virtio-crypto-pci: Tidy up virtio_crypto_pci_realize()
  qemu-option: Check return value instead of @err where convenient
  qemu-option: Make uses of find_desc_by_name() more similar
  qemu-option: Factor out helper find_default_by_name()
  qemu-option: Simplify around find_default_by_name()
  qemu-option: Factor out helper opt_create()
  qemu-option: Replace opt_set() by cleaner opt_validate()
  qemu-option: Make functions taking Error ** return bool, not void
  qemu-option: Use returned bool to check for failure
  block: Avoid error accumulation in bdrv_img_create()
  hmp: Eliminate a variable in hmp_migrate_set_parameter()
  qapi: Make visitor functions taking Error ** return bool, not void
  qapi: Use returned bool to check for failure, Coccinelle part
  qapi: Use returned bool to check for failure, manual part
  block/parallels: Simplify parallels_open() after previous commit
  s390x/pci: Fix harmless mistake in zpci's property fid's setter
  qom: Use error_reportf_err() instead of g_printerr() in examples
  qom: Rename qdev_get_type() to object_get_type()
  qom: Crash more nicely on object_property_get_link() failure
  qom: Don't handle impossible object_property_get_link() failure
  qom: Use return values to check for error where that's simpler
  qom: Put name parameter before value / visitor parameter
  qom: Make functions taking Error ** return bool, not void
  qom: Use returned bool to check for failure, Coccinelle part
  qom: Use returned bool to check for failure, manual part
  qom: Make functions taking Error ** return bool, not 0/-1
  qdev: Make functions taking Error ** return bool, not void
  qdev: Use returned bool to check for failure, Coccinelle part
  error: Avoid unnecessary error_propagate() after error_setg()
  error: Eliminate error_propagate() with Coccinelle, part 1
  error: Eliminate error_propagate() with Coccinelle, part 2
  error: Eliminate error_propagate() manually
  error: Reduce unnecessary error propagation
  qapi: Smooth another visitor error checking pattern
  qapi: Smooth visitor error checking in generated code
  qapi: Purge error_propagate() from QAPI core
  error: Avoid error_propagate() after migrate_add_blocker()
  qemu-img: Ignore Error objects where the return value suffices
  qdev: Ignore Error objects where the return value suffices
  hmp: Ignore Error objects where the return value suffices

 docs/devel/qapi-code-gen.txt             | 103 ++++-----
 include/hw/audio/pcspk.h                 |   2 +-
 include/hw/qdev-properties.h             |   4 +-
 include/qapi/clone-visitor.h             |   8 +-
 include/qapi/error.h                     |  45 +++-
 include/qapi/visitor-impl.h              |  26 +--
 include/qapi/visitor.h                   | 102 +++++----
 include/qemu/option.h                    |  16 +-
 include/qom/object.h                     | 105 +++++----
 include/qom/object_interfaces.h          |  12 +-
 include/qom/qom-qobject.h                |   9 +-
 accel/kvm/kvm-all.c                      |  55 +++--
 accel/tcg/tcg-all.c                      |   5 +-
 audio/audio_legacy.c                     |  15 +-
 backends/cryptodev-vhost-user.c          |   3 +-
 backends/cryptodev.c                     |  16 +-
 backends/hostmem-file.c                  |  22 +-
 backends/hostmem-memfd.c                 |  18 +-
 backends/hostmem.c                       |  33 ++-
 backends/rng.c                           |   2 +-
 backends/tpm/tpm_util.c                  |   5 +-
 block.c                                  |  21 +-
 block/blkdebug.c                         |   9 +-
 block/blklogwrites.c                     |   4 +-
 block/blkverify.c                        |   4 +-
 block/crypto.c                           |   5 +-
 block/curl.c                             |   5 +-
 block/file-posix.c                       |  16 +-
 block/file-win32.c                       |   8 +-
 block/gluster.c                          |  17 +-
 block/iscsi.c                            |   4 +-
 block/nbd.c                              |  10 +-
 block/nfs.c                              |   7 +-
 block/parallels.c                        |  29 +--
 block/qcow.c                             |  16 +-
 block/qcow2.c                            |  21 +-
 block/qed.c                              |  10 +-
 block/quorum.c                           |  19 +-
 block/raw-format.c                       |   5 +-
 block/rbd.c                              |   7 +-
 block/replication.c                      |  19 +-
 block/sheepdog.c                         |  16 +-
 block/ssh.c                              |  11 +-
 block/throttle-groups.c                  |  31 +--
 block/throttle.c                         |   5 +-
 block/vdi.c                              |  13 +-
 block/vhdx.c                             |  15 +-
 block/vmdk.c                             |  13 +-
 block/vpc.c                              |  19 +-
 block/vvfat.c                            |  10 +-
 block/vxhs.c                             |  15 +-
 blockdev.c                               |  40 ++--
 bootdevice.c                             |  13 +-
 chardev/char.c                           |   6 +-
 contrib/ivshmem-server/main.c            |   4 +-
 crypto/secret.c                          |   2 +-
 crypto/secret_keyring.c                  |   2 +-
 crypto/tlscredsanon.c                    |   2 +-
 crypto/tlscredspsk.c                     |   2 +-
 crypto/tlscredsx509.c                    |   2 +-
 dump/dump.c                              |   7 +-
 hw/acpi/core.c                           |  19 +-
 hw/acpi/cpu_hotplug.c                    |   4 +-
 hw/acpi/ich9.c                           |   2 +-
 hw/acpi/piix4.c                          |   2 +-
 hw/arm/allwinner-a10.c                   |  27 +--
 hw/arm/armsse.c                          | 208 ++++++------------
 hw/arm/armv7m.c                          |  47 ++---
 hw/arm/aspeed.c                          |  24 +--
 hw/arm/aspeed_ast2600.c                  | 124 ++++-------
 hw/arm/aspeed_soc.c                      |  85 +++-----
 hw/arm/bcm2835_peripherals.c             |  81 ++-----
 hw/arm/bcm2836.c                         |  35 +--
 hw/arm/cubieboard.c                      |  14 +-
 hw/arm/digic.c                           |  18 +-
 hw/arm/digic_boards.c                    |   3 +-
 hw/arm/exynos4210.c                      |  13 +-
 hw/arm/fsl-imx25.c                       |  58 ++---
 hw/arm/fsl-imx31.c                       |  34 +--
 hw/arm/fsl-imx6.c                        |  85 +++-----
 hw/arm/fsl-imx6ul.c                      |  24 +--
 hw/arm/fsl-imx7.c                        |  31 ++-
 hw/arm/highbank.c                        |  12 +-
 hw/arm/integratorcp.c                    |   2 +-
 hw/arm/microbit.c                        |   4 +-
 hw/arm/mps2-tz.c                         |  31 ++-
 hw/arm/mps2.c                            |  12 +-
 hw/arm/msf2-soc.c                        |  29 +--
 hw/arm/musca.c                           |  18 +-
 hw/arm/musicpal.c                        |   4 +-
 hw/arm/nrf51_soc.c                       |  36 +---
 hw/arm/orangepi.c                        |  13 +-
 hw/arm/raspi.c                           |   2 +-
 hw/arm/realview.c                        |   6 +-
 hw/arm/sbsa-ref.c                        |  16 +-
 hw/arm/stellaris.c                       |   4 +-
 hw/arm/stm32f205_soc.c                   |  37 +---
 hw/arm/stm32f405_soc.c                   |  48 ++---
 hw/arm/versatilepb.c                     |   4 +-
 hw/arm/vexpress.c                        |   8 +-
 hw/arm/virt.c                            |  44 ++--
 hw/arm/xilinx_zynq.c                     |   6 +-
 hw/arm/xlnx-versal-virt.c                |   8 +-
 hw/arm/xlnx-versal.c                     |  30 ++-
 hw/arm/xlnx-zcu102.c                     |   8 +-
 hw/arm/xlnx-zynqmp.c                     | 117 ++++------
 hw/block/fdc.c                           |  12 +-
 hw/block/xen-block.c                     |  30 +--
 hw/char/serial-pci-multi.c               |   5 +-
 hw/char/serial-pci.c                     |   5 +-
 hw/char/serial.c                         |  10 +-
 hw/core/bus.c                            |  12 +-
 hw/core/cpu.c                            |   3 +-
 hw/core/machine.c                        |   5 +-
 hw/core/numa.c                           |  55 ++---
 hw/core/platform-bus.c                   |   6 +-
 hw/core/qdev-properties-system.c         |  32 +--
 hw/core/qdev-properties.c                |  93 +++-----
 hw/core/qdev.c                           |  16 +-
 hw/cpu/a15mpcore.c                       |   5 +-
 hw/cpu/a9mpcore.c                        |  21 +-
 hw/cpu/arm11mpcore.c                     |  17 +-
 hw/cpu/core.c                            |  10 +-
 hw/cpu/realview_mpcore.c                 |   9 +-
 hw/display/bcm2835_fb.c                  |   8 +-
 hw/display/virtio-gpu-base.c             |   5 +-
 hw/display/virtio-gpu-pci.c              |  11 +-
 hw/display/virtio-vga.c                  |  10 +-
 hw/dma/bcm2835_dma.c                     |   9 +-
 hw/dma/sparc32_dma.c                     |   6 +-
 hw/dma/xilinx_axidma.c                   |   4 +-
 hw/gpio/aspeed_gpio.c                    |   5 +-
 hw/gpio/bcm2835_gpio.c                   |  15 +-
 hw/hyperv/vmbus.c                        |   5 +-
 hw/i386/pc.c                             |  48 ++---
 hw/i386/pc_piix.c                        |   4 +-
 hw/i386/pc_q35.c                         |  28 +--
 hw/i386/x86.c                            |   7 +-
 hw/ide/qdev.c                            |   7 +-
 hw/intc/apic_common.c                    |   5 +-
 hw/intc/arm_gic_kvm.c                    |   4 +-
 hw/intc/arm_gicv3_its_kvm.c              |   5 +-
 hw/intc/arm_gicv3_kvm.c                  |   4 +-
 hw/intc/armv7m_nvic.c                    |   9 +-
 hw/intc/nios2_iic.c                      |   8 +-
 hw/intc/pnv_xive.c                       |  17 +-
 hw/intc/realview_gic.c                   |   5 +-
 hw/intc/spapr_xive.c                     |  17 +-
 hw/intc/xics.c                           |   9 +-
 hw/intc/xics_kvm.c                       |   4 +-
 hw/intc/xive.c                           |  17 +-
 hw/isa/piix4.c                           |   5 +-
 hw/m68k/q800.c                           |   4 +-
 hw/mem/nvdimm.c                          |  30 +--
 hw/mem/pc-dimm.c                         |  18 +-
 hw/microblaze/petalogix_ml605_mmu.c      |  24 +--
 hw/microblaze/petalogix_s3adsp1800_mmu.c |   2 +-
 hw/microblaze/xlnx-zynqmp-pmu.c          |  39 ++--
 hw/mips/boston.c                         |   4 +-
 hw/mips/cps.c                            |  41 ++--
 hw/mips/jazz.c                           |   4 +-
 hw/mips/malta.c                          |   4 +-
 hw/misc/aspeed_sdmc.c                    |   8 +-
 hw/misc/bcm2835_mbox.c                   |   9 +-
 hw/misc/bcm2835_property.c               |  17 +-
 hw/misc/iotkit-sysctl.c                  |   2 +-
 hw/misc/ivshmem.c                        |   4 +-
 hw/misc/macio/cuda.c                     |   5 +-
 hw/misc/macio/macio.c                    |  35 ++-
 hw/misc/macio/pmu.c                      |   5 +-
 hw/misc/pca9552.c                        |   5 +-
 hw/misc/tmp105.c                         |   5 +-
 hw/misc/tmp421.c                         |   5 +-
 hw/net/ne2000-isa.c                      |   7 +-
 hw/net/virtio-net.c                      |   7 +-
 hw/net/xilinx_axienet.c                  |   4 +-
 hw/pci-host/pnv_phb3.c                   |  33 ++-
 hw/pci-host/pnv_phb4.c                   |   9 +-
 hw/pci-host/pnv_phb4_pec.c               |   9 +-
 hw/pci-host/prep.c                       |   4 +-
 hw/ppc/e500.c                            |   5 +-
 hw/ppc/mac_newworld.c                    |  10 +-
 hw/ppc/mac_oldworld.c                    |   4 +-
 hw/ppc/pnv.c                             | 171 +++++++--------
 hw/ppc/pnv_core.c                        |   4 +-
 hw/ppc/pnv_psi.c                         |  22 +-
 hw/ppc/rs6000_mc.c                       |   9 +-
 hw/ppc/spapr.c                           |  77 +++----
 hw/ppc/spapr_caps.c                      |  15 +-
 hw/ppc/spapr_cpu_core.c                  |  15 +-
 hw/ppc/spapr_drc.c                       |  16 +-
 hw/ppc/spapr_hcall.c                     |   3 +-
 hw/ppc/spapr_irq.c                       |  11 +-
 hw/ppc/spapr_pci.c                       |  16 +-
 hw/ppc/spapr_pci_nvlink2.c               |   3 +-
 hw/riscv/opentitan.c                     |  13 +-
 hw/riscv/sifive_e.c                      |  10 +-
 hw/riscv/sifive_u.c                      |  11 +-
 hw/riscv/spike.c                         |   4 +-
 hw/riscv/virt.c                          |   4 +-
 hw/rx/rx-gdbsim.c                        |  12 +-
 hw/s390x/css.c                           |   5 +-
 hw/s390x/event-facility.c                |  13 +-
 hw/s390x/ipl.c                           |  27 ++-
 hw/s390x/s390-pci-bus.c                  |  14 +-
 hw/s390x/s390-skeys.c                    |   2 +-
 hw/s390x/s390-stattrib.c                 |   2 +-
 hw/s390x/s390-virtio-ccw.c               |  20 +-
 hw/s390x/sclp.c                          |  13 +-
 hw/s390x/virtio-ccw-crypto.c             |  10 +-
 hw/s390x/virtio-ccw-rng.c                |   8 +-
 hw/scsi/scsi-bus.c                       |  15 +-
 hw/scsi/vhost-scsi.c                     |   4 +-
 hw/sd/aspeed_sdhci.c                     |  15 +-
 hw/sd/sd.c                               |   3 +-
 hw/sd/ssi-sd.c                           |  11 +-
 hw/smbios/smbios.c                       |  33 +--
 hw/sparc/sun4m.c                         |   2 +-
 hw/sparc64/sun4u.c                       |   2 +-
 hw/usb/bus.c                             |   7 +-
 hw/usb/dev-storage.c                     |   9 +-
 hw/usb/hcd-dwc2.c                        |   9 +-
 hw/vfio/pci-quirks.c                     |   5 +-
 hw/vfio/pci.c                            |  10 +-
 hw/virtio/virtio-balloon.c               |  17 +-
 hw/virtio/virtio-crypto-pci.c            |   9 +-
 hw/virtio/virtio-iommu-pci.c             |   4 +-
 hw/virtio/virtio-pmem-pci.c              |   2 +-
 hw/virtio/virtio-rng-pci.c               |   8 +-
 hw/virtio/virtio-rng.c                   |  10 +-
 hw/xen/xen_pt_config_init.c              |   3 +-
 iothread.c                               |  18 +-
 linux-user/syscall.c                     |   2 +-
 monitor/hmp-cmds.c                       |  11 +-
 monitor/monitor.c                        |  21 +-
 net/colo-compare.c                       |  26 +--
 net/dump.c                               |  13 +-
 net/filter-buffer.c                      |  13 +-
 net/filter.c                             |   2 +-
 net/net.c                                |  10 +-
 net/tap.c                                |   6 +-
 qapi/opts-visitor.c                      |  58 ++---
 qapi/qapi-clone-visitor.c                |  67 +++---
 qapi/qapi-dealloc-visitor.c              |  27 ++-
 qapi/qapi-visit-core.c                   | 197 ++++++++---------
 qapi/qobject-input-visitor.c             | 109 ++++++----
 qapi/qobject-output-visitor.c            |  27 ++-
 qapi/string-input-visitor.c              |  67 +++---
 qapi/string-output-visitor.c             |  32 +--
 qdev-monitor.c                           |  38 ++--
 qemu-img.c                               |  23 +-
 qga/commands-posix.c                     |   4 +-
 qga/commands-win32.c                     |  22 +-
 qom/object.c                             | 235 ++++++++++-----------
 qom/object_interfaces.c                  |  30 +--
 qom/qom-hmp-cmds.c                       |   2 +-
 qom/qom-qmp-cmds.c                       |   2 +-
 qom/qom-qobject.c                        |  14 +-
 softmmu/vl.c                             |  17 +-
 target/arm/cpu64.c                       |  15 +-
 target/arm/monitor.c                     |   7 +-
 target/i386/cpu.c                        |  98 +++------
 target/ppc/compat.c                      |   5 +-
 target/ppc/translate_init.inc.c          |   2 +-
 target/s390x/cpu_models.c                |  17 +-
 target/sparc/cpu.c                       |   5 +-
 tpm.c                                    |   5 +-
 ui/console.c                             |   4 +-
 ui/vnc.c                                 |   2 +-
 util/main-loop.c                         |   4 +-
 util/qemu-config.c                       |  25 +--
 util/qemu-option.c                       | 258 ++++++++++++-----------
 scripts/qapi/commands.py                 |  22 +-
 scripts/qapi/visit.py                    | 107 ++++------
 274 files changed, 2349 insertions(+), 3524 deletions(-)

-- 
2.26.2



^ permalink raw reply	[flat|nested] 88+ messages in thread

end of thread, other threads:[~2020-07-04 16:17 UTC | newest]

Thread overview: 88+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-02 15:49 [PATCH v2 00/44] Less clumsy error checking Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 01/44] error: Improve examples in error.h's big comment Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 02/44] error: Document Error API usage rules Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 03/44] qdev: Use returned bool to check for qdev_realize() etc. failure Markus Armbruster
2020-07-02 16:23   ` Eric Blake
2020-07-02 18:57   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 04/44] macio: Tidy up error handling in macio_newworld_realize() Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 05/44] virtio-crypto-pci: Tidy up virtio_crypto_pci_realize() Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 06/44] qemu-option: Check return value instead of @err where convenient Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 07/44] qemu-option: Make uses of find_desc_by_name() more similar Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 08/44] qemu-option: Factor out helper find_default_by_name() Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 09/44] qemu-option: Simplify around find_default_by_name() Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 10/44] qemu-option: Factor out helper opt_create() Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 11/44] qemu-option: Replace opt_set() by cleaner opt_validate() Markus Armbruster
2020-07-02 19:49   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 12/44] qemu-option: Make functions taking Error ** return bool, not void Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 13/44] qemu-option: Use returned bool to check for failure Markus Armbruster
2020-07-02 16:25   ` Eric Blake
2020-07-03 12:15   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 14/44] block: Avoid error accumulation in bdrv_img_create() Markus Armbruster
2020-07-02 16:26   ` Eric Blake
2020-07-03  6:46     ` Markus Armbruster
2020-07-03 12:24   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 15/44] hmp: Eliminate a variable in hmp_migrate_set_parameter() Markus Armbruster
2020-07-03 12:29   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 16/44] qapi: Make visitor functions taking Error ** return bool, not void Markus Armbruster
2020-07-02 16:35   ` Eric Blake
2020-07-03 14:32   ` Vladimir Sementsov-Ogievskiy
2020-07-04 13:19     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 17/44] qapi: Use returned bool to check for failure, Coccinelle part Markus Armbruster
2020-07-02 17:28   ` Eric Blake
2020-07-03 14:56   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 18/44] qapi: Use returned bool to check for failure, manual part Markus Armbruster
2020-07-03 15:10   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 19/44] block/parallels: Simplify parallels_open() after previous commit Markus Armbruster
2020-07-03 15:29   ` Vladimir Sementsov-Ogievskiy
2020-07-04 13:28     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 20/44] s390x/pci: Fix harmless mistake in zpci's property fid's setter Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 21/44] qom: Use error_reportf_err() instead of g_printerr() in examples Markus Armbruster
2020-07-03 15:33   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 22/44] qom: Rename qdev_get_type() to object_get_type() Markus Armbruster
2020-07-03 15:35   ` Vladimir Sementsov-Ogievskiy
2020-07-02 15:49 ` [PATCH v2 23/44] qom: Crash more nicely on object_property_get_link() failure Markus Armbruster
2020-07-02 17:29   ` Eric Blake
2020-07-03 15:43   ` Vladimir Sementsov-Ogievskiy
2020-07-04 16:15     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 24/44] qom: Don't handle impossible " Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 25/44] qom: Use return values to check for error where that's simpler Markus Armbruster
2020-07-03 16:06   ` Vladimir Sementsov-Ogievskiy
2020-07-04 14:06     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 26/44] qom: Put name parameter before value / visitor parameter Markus Armbruster
2020-07-03 18:05   ` Vladimir Sementsov-Ogievskiy
2020-07-03 18:06     ` Vladimir Sementsov-Ogievskiy
2020-07-04 16:02       ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 27/44] qom: Make functions taking Error ** return bool, not void Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 28/44] qom: Use returned bool to check for failure, Coccinelle part Markus Armbruster
2020-07-02 17:31   ` Eric Blake
2020-07-02 15:49 ` [PATCH v2 29/44] qom: Use returned bool to check for failure, manual part Markus Armbruster
2020-07-02 17:32   ` Eric Blake
2020-07-02 15:49 ` [PATCH v2 30/44] qom: Make functions taking Error ** return bool, not 0/-1 Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 31/44] qdev: Make functions taking Error ** return bool, not void Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 32/44] qdev: Use returned bool to check for failure, Coccinelle part Markus Armbruster
2020-07-02 17:32   ` Eric Blake
2020-07-02 15:49 ` [PATCH v2 33/44] error: Avoid unnecessary error_propagate() after error_setg() Markus Armbruster
2020-07-02 17:43   ` Eric Blake
2020-07-03  6:53     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 34/44] error: Eliminate error_propagate() with Coccinelle, part 1 Markus Armbruster
2020-07-02 18:02   ` Eric Blake
2020-07-03  6:55     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 35/44] error: Eliminate error_propagate() with Coccinelle, part 2 Markus Armbruster
2020-07-02 18:06   ` Eric Blake
2020-07-03  6:59     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 36/44] error: Eliminate error_propagate() manually Markus Armbruster
2020-07-02 18:25   ` Eric Blake
2020-07-03  6:59     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 37/44] error: Reduce unnecessary error propagation Markus Armbruster
2020-07-02 18:27   ` Eric Blake
2020-07-03  7:09     ` Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 38/44] qapi: Smooth another visitor error checking pattern Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 39/44] qapi: Smooth visitor error checking in generated code Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 40/44] qapi: Purge error_propagate() from QAPI core Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 41/44] error: Avoid error_propagate() after migrate_add_blocker() Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 42/44] qemu-img: Ignore Error objects where the return value suffices Markus Armbruster
2020-07-02 15:49 ` [PATCH v2 43/44] qdev: " Markus Armbruster
2020-07-02 15:50 ` [PATCH v2 44/44] hmp: " Markus Armbruster
2020-07-02 15:54 ` [PATCH v2 00/44] Less clumsy error checking Markus Armbruster
2020-07-02 16:56 ` no-reply
2020-07-02 16:58 ` no-reply

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).