qemu-devel
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a

From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one
Date: Thu, 3 May 2018 15:02:59 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 
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. 



Signed-off-by: Michael S. Tsirkin <address@hidden>
---
  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
reply via email to
[Prev in Thread] Current Thread [Next in Thread]