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

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to