[libvirt] [PATCH v6 1/8] snapshot: Split domain forward typedefs into new file

Eric Blake eblake at redhat.com
Fri Mar 15 13:49:43 UTC 2019


On 3/15/19 8:21 AM, Eric Blake wrote:
> On 3/15/19 7:46 AM, Ján Tomko wrote:
>> [ not sure how John Snow got on the CC list. Hi, John! ]
>>
>> s/snapshot/conf/ in the summary
> 
> Sure. It's related to my work on snapshots, but independent enough (and
> doesn't even touch snapshot_conf).
> 

>>> # include "internal.h"
>>> +# include "virconftypes.h"
>>
>> The files in src conf do not use the vir prefix.
>> How about "conf_types.h"?
> 
> Naming is hard. I can live with that.

Actually, I'm leaning towards keeping the "vir" prefix (to avoid any
chance of ever including a wrong file from outside of libvirt), and as
you noticed we DO have other (newer) files in src/conf/ with a vir prefix.

-- 
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/2747a47a/attachment-0001.sig>


More information about the libvir-list mailing list