[libvirt] [PATCH v3 09/14] virsh: Create macros for common "pool" options
Andrea Bolognani
abologna at redhat.com
Tue Jan 12 09:22:48 UTC 2016
On Mon, 2016-01-11 at 18:03 -0500, John Ferlan wrote:
> I've also lift/applied the "helpstr" to each of the # define
> VIRSH_COMMON_OPT_*'s in virsh.h (meaning virsh.h is not included in
> po/POTFILES.in either). That also picks up a couple more domain options
> from virsh-domain.c which use different helpstr's.
What's the advantage of doing so? Now we have two separate, identical
definitions for eg. VIRSH_COMMON_OPT_DOMAIN_FULL...
> > Is there any actual difference between VSH_OT_STRING and
> > VSH_OT_DATA? Can we convert this to VSH_OT_DATA and rename it to
> > VIRSH_COMMON_OPT_POOL_OPTIONAL or something like that? Or maybe
> > have *this* as VIRSH_COMMON_OPT_POOL and rename the one
> > describing a required option to VIRSH_COMMON_OPT_POOL_REQ?
>
> OT_DATA and OT_STRING are handled differently... STRING is never
> OFLAG_REQ and DATA is always OFLAG_REQ.
Right. I always forget that :(
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team
More information about the libvir-list
mailing list