On Thu, May 03, 2018 at 03:02:59PM -0500, Eric Blake wrote: > On 05/03/2018 02:50 PM, Michael S. Tsirkin wrote: > > we just need a struct name, let's add a forward > > declaration instead of an include. > > Reasonable. > > > > > We also use size_t, so add stddef.h > > Why? osdep.h already does this, and ALL .c files that use include/ should be > including osdep.h prior to any other in-tree .h file. So size_t should > already be in scope by the time this header is included.
It seemed cleaner to have the file self-contained. OK, I will drop this. There's a small number of files which do not include qemu/osdep.h. Might be worth fixing. > > > > Signed-off-by: Michael S. Tsirkin <m...@redhat.com> > > --- > > include/migration/vmstate.h | 4 +++- > > migration/savevm.c | 1 + > > 2 files changed, 4 insertions(+), 1 deletion(-) > -- > Eric Blake, Principal Software Engineer > Red Hat, Inc. +1-919-301-3266 > Virtualization: qemu.org | libvirt.org