Re: [Qemu-devel] [PATCH 07/18] Eliminate qapi/qmp/types.h

[Eric Blake]

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 <address@hidden>
> ---
>  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 <address@hidden>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

signature.asc
Description: OpenPGP digital signature