Hi all! This is the first part of the bit series, which contains mostly simple cleanups.
v6 was sent in separate (I'm sorry for inconvenience) v7: by Markus review (and with his prepared fixups, thanks a lot!): - don't rename Error** paramters - switch to Error *const * where appropriate last patch is new and replaces "nbd: well form nbd_iter_channel_error errp handler" Vladimir Sementsov-Ogievskiy (21): hw/core/loader-fit: fix freeing errp in fit_load_fdt net/net: Clean up variable shadowing in net_client_init() error: rename errp to errp_in where it is IN-argument hmp: drop Error pointer indirection in hmp_handle_error vnc: drop Error pointer indirection in vnc_client_io_error qdev-monitor: well form error hint helpers ppc: well form kvmppc_hint_smt_possible error hint helper 9pfs: well form error hint helpers hw/core/qdev: cleanup Error ** variables block/snapshot: rename Error ** parameter to more common errp hw/i386/amd_iommu: rename Error ** parameter to more common errp qga: rename Error ** parameter to more common errp monitor/qmp-cmds: rename Error ** parameter to more common errp hw/s390x: rename Error ** parameter to more common errp hw/sd: drop extra whitespace in sdhci_sysbus_realize() header hw/tpm: rename Error ** parameter to more common errp hw/usb: rename Error ** parameter to more common errp include/qom/object.h: rename Error ** parameter to more common errp backends/cryptodev: drop local_err from cryptodev_backend_complete() hw/vfio/ap: drop local_err from vfio_ap_realize nbd: assert that Error** is not NULL in nbd_iter_channel_error Cc: "Gonglei (Arei)" <[email protected]> Cc: Eric Blake <[email protected]> Cc: Kevin Wolf <[email protected]> Cc: Max Reitz <[email protected]> Cc: "Marc-André Lureau" <[email protected]> Cc: Greg Kurz <[email protected]> Cc: Paul Burton <[email protected]> Cc: Aleksandar Rikalo <[email protected]> Cc: Paolo Bonzini <[email protected]> Cc: "Daniel P. Berrangé" <[email protected]> Cc: Eduardo Habkost <[email protected]> Cc: Richard Henderson <[email protected]> Cc: "Michael S. Tsirkin" <[email protected]> Cc: Marcel Apfelbaum <[email protected]> Cc: David Gibson <[email protected]> Cc: Cornelia Huck <[email protected]> Cc: David Hildenbrand <[email protected]> Cc: Halil Pasic <[email protected]> Cc: Christian Borntraeger <[email protected]> Cc: "Philippe Mathieu-Daudé" <[email protected]> Cc: Stefan Berger <[email protected]> Cc: Gerd Hoffmann <[email protected]> Cc: Tony Krowiak <[email protected]> Cc: Pierre Morel <[email protected]> Cc: Alex Williamson <[email protected]> Cc: Markus Armbruster <[email protected]> Cc: Michael Roth <[email protected]> Cc: "Dr. David Alan Gilbert" <[email protected]> Cc: Jason Wang <[email protected]> Cc: [email protected] Cc: [email protected] Cc: [email protected] Cc: [email protected] include/block/snapshot.h | 2 +- include/monitor/hmp.h | 2 +- include/qapi/error.h | 6 +- include/qom/object.h | 4 +- target/ppc/kvm_ppc.h | 4 +- ui/vnc.h | 2 +- backends/cryptodev.c | 11 +-- block/nbd.c | 1 + block/snapshot.c | 4 +- dump/dump-hmp-cmds.c | 4 +- hw/9pfs/9p-local.c | 2 +- hw/9pfs/9p-proxy.c | 2 +- hw/core/loader-fit.c | 5 +- hw/core/machine-hmp-cmds.c | 6 +- hw/core/qdev.c | 28 ++++--- hw/i386/amd_iommu.c | 14 ++-- hw/ppc/spapr.c | 2 +- hw/s390x/event-facility.c | 2 +- hw/s390x/s390-stattrib.c | 3 +- hw/sd/sdhci.c | 2 +- hw/tpm/tpm_emulator.c | 8 +- hw/usb/dev-network.c | 2 +- hw/vfio/ap.c | 9 +-- monitor/hmp-cmds.c | 155 ++++++++++++++++++------------------- monitor/qmp-cmds.c | 2 +- net/net.c | 17 ++-- qdev-monitor.c | 16 ++-- qga/commands-posix.c | 2 +- qga/commands-win32.c | 2 +- qga/commands.c | 12 +-- qom/qom-hmp-cmds.c | 4 +- target/ppc/kvm.c | 2 +- ui/vnc.c | 20 ++--- util/error.c | 6 +- 34 files changed, 173 insertions(+), 190 deletions(-) -- 2.21.0
