On 01/30/2018 04:21 AM, Markus Armbruster wrote:
> qapi/qmp/types.h is a convenience header to include a number of
> qapi/qmp/ headers.  Since we rarely need all of the headers
> qapi/qmp/types.h includes, we bypass it most of the time.  Most of the
> places that use it don't need all the headers, either.
> 
> Include the necessary headers directly, and drop qapi/qmp/types.h.
> 
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
>  block/qapi.c                        |  3 ++-
>  block/qcow2.c                       |  3 ++-
>  blockdev.c                          |  2 +-
>  hw/pci/pcie_aer.c                   |  1 -
>  hw/watchdog/watchdog.c              |  1 -
>  include/qapi/qmp/types.h            | 24 ------------------------

Goodbye!  Glad to be rid of you!

Reviewed-by: Eric Blake <ebl...@redhat.com>

-- 
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