On 06/08/2017 08:39 AM, Eduardo Habkost wrote: > This series fixes and simplifies some of the existing code that > assigns to *errp directly. > > Eduardo Habkost (5): > xilinx: Fix error handling > block: Don't try to set *errp directly > websock: Don't try to set *errp directly > migration: Don't try to set *errp directly > vnc: No need for Error** parameter at vnc_client_io_error()
For 1-4: Reviewed-by: Eric Blake <ebl...@redhat.com> I'm not sure about 5 > > ui/vnc.h | 2 +- > block.c | 8 +++----- > hw/dma/xilinx_axidma.c | 4 +--- > hw/net/xilinx_axienet.c | 4 +--- > io/channel-websock.c | 4 ++-- > migration/migration.c | 2 +- > ui/vnc.c | 13 +++++-------- > 7 files changed, 14 insertions(+), 23 deletions(-) > -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature