Re: [Qemu-devel] [PATCH 14/15] Don't include qemu-common.h in headers

[Peter Maydell]

On 15 March 2016 at 17:44, Paolo Bonzini <address@hidden> wrote:
> Well, osdep.h is the new qemu-common.h.  At least it doesn't include
> other^Wtoo many other QEMU header files, but it does have a lot of
> prototypes that are even more out of place than fprintf_function.

Yes, those should be moved to somewhere else, they're in the
wrong place (as noted in the comment at the top of osdep.h).

The comment at the top of osdep.h describes the plan:

 * In an ideal world this header would contain only:
 *  (1) things which everybody needs
 *  (2) things without which code would work on most platforms but
 *      fail to compile or misbehave on a minority of host OSes

and I think we should be aiming to move towards the ideal world
rather than away from it.

thanks
-- PMM