[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