[PATCH] virConnectDomainEventRegisterAny: correct docs

Daniel P. Berrangé berrange at redhat.com
Mon Mar 28 14:34:25 UTC 2022


On Mon, Mar 28, 2022 at 04:21:15PM +0200, Michal Prívozník wrote:
> On 3/25/22 17:03, John Levon wrote:
> > The callback ID can be zero, not necessarily positive; correct the
> > comment to reflect this.
> > 
> > Signed-off-by: John Levon <levon at movementarian.org>
> > ---
> >  src/libvirt-domain.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
> > index a197618673..53efb60406 100644
> > --- a/src/libvirt-domain.c
> > +++ b/src/libvirt-domain.c
> > @@ -9486,7 +9486,7 @@ virDomainMigrateStartPostCopy(virDomainPtr domain,
> >   * The reference can be released once the object is no longer required
> >   * by calling virDomainFree().
> >   *
> > - * The return value from this method is a positive integer identifier
> > + * The return value from this method is a non-negative integer identifier
> >   * for the callback. To unregister a callback, this callback ID should
> >   * be passed to the virConnectDomainEventDeregisterAny() method.
> >   *
> 
> Reviewed-by: Michal Privoznik <mprivozn at redhat.com>
> 
> and pushed. Congratulations on your first libvirt contribution.

...in a little while.

John used to help maintain us libvirt on Solaris back in the very early
days when we were Xen focused and KVM was barely starting to get off
the ground :-)

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|


More information about the libvir-list mailing list