On 7/20/21 2:54 PM, Markus Armbruster wrote:
> Most callers check the return value.  Some check whether it set an
> error.  Functionally equivalent, but the former tends to be easier on
> the eyes, so do that everywhere.
> 
> Prior art: commit c6ecec43b2 "qemu-option: Check return value instead
> of @err where convenient".
> 
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
>  backends/tpm/tpm_emulator.c | 3 +--
>  hw/display/qxl.c            | 8 ++++----
>  hw/virtio/vhost.c           | 2 +-
>  target/i386/kvm/kvm.c       | 6 +++---
>  target/i386/nvmm/nvmm-all.c | 3 +--
>  target/i386/whpx/whpx-all.c | 3 +--
>  6 files changed, 11 insertions(+), 14 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>


Reply via email to