[libvirt] [PATCH v6 5/8] snapshot: Break out virDomainSnapshotObjList into its own file

Eric Blake eblake at redhat.com
Fri Mar 15 14:12:15 UTC 2019


On 3/15/19 12:02 AM, Eric Blake wrote:
> snapshot_conf.h was mixing three separate types: the snapshot
> definition, the snapshot object, and the snapshot object list.
> Separate out the snapshot object list code into its own file, and
> update includes for affected clients.
> 
> This is just code motion, but done in preparation of sharing a lot of
> the object list code with checkpoints.
> 
> Signed-off-by: Eric Blake <eblake at redhat.com>
> ---

> +++ b/src/conf/virdomainsnapshotobjlist.h
> @@ -0,0 +1,74 @@

> +#ifndef LIBVIRT_VIRDOMAINSNAPSHOTOBJLIST_H
> +# define LIBVIRT_VIRDOMAINSNAPSHOTOBJLIST_H
> +
> +# include "internal.h"
> +# include "virdomainsnapshotobj.h"
> +# include "capabilities.h"

This one felt a bit unclean. I'll submit a patch 1.5/8 that moves the
typedefs from capabilities.h int virconftypes.h, then fix 2/8 to sort
those into place as well.

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190315/713e7173/attachment-0001.sig>


More information about the libvir-list mailing list