On Fri, May 04, 2018 at 17:25:31 +0800, Lin Ma wrote: > Signed-off-by: Lin Ma <lma@xxxxxxxx> > --- > include/libvirt/libvirt-domain.h | 201 +++++++++++++++++++++++++++++++++++++++ > tools/virsh-domain.c | 50 +++++----- > 2 files changed, 226 insertions(+), 25 deletions(-) > > diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h > index 12fd34037e..f7dd510f7f 100644 > --- a/include/libvirt/libvirt-domain.h > +++ b/include/libvirt/libvirt-domain.h > @@ -4424,6 +4424,207 @@ typedef enum { > } virDomainEventID; > > > +/** > + * VIR_DOMAIN_EVENT_LIFECYCLE: > + * > + * Macro for the event name "lifecycle" > + */ > + > +# define VIR_DOMAIN_EVENT_LIFECYCLE "lifecycle" This would make all of these public API and I'm not persuaded this is a good idea. Specifically since they are only used in virsh. The library always refers to the events by number. Also some of the events have multiple implementations which don't usually need to be referred to publically depending on the use case of the event. Virsh implements all of them just for the sake of completness. NACK
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list