On Thu, Jul 19, 2012 at 01:12:33PM -0600, Eric Blake wrote:
> On 07/19/2012 09:04 AM, Daniel P. Berrange wrote:
> > From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
> >
> > Use a driver close callback to trigger shutdown of the
> > events demo program
> >
> > Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
> > ---
> > examples/domain-events/events-c/event-test.c | 33 ++++++++++++++++++++++++--
> > 1 file changed, 31 insertions(+), 2 deletions(-)
> >
> > diff --git a/examples/domain-events/events-c/event-test.c b/examples/domain-events/events-c/event-test.c
> > index ef6e77a..f2427ad 100644
> > --- a/examples/domain-events/events-c/event-test.c
> > +++ b/examples/domain-events/events-c/event-test.c
> > @@ -16,6 +16,8 @@
> > # define ATTRIBUTE_UNUSED __attribute__((__unused__))
> > #endif
> >
> > +int run = 1;
>
> bool? Then again, <stdbool.h> isn't available everywhere, and I think
> our examples can work with C89 even though the rest of our project
> requires C99.
>
> > @@ -380,7 +407,6 @@ void usage(const char *pname)
> > printf("%s uri\n", pname);
> > }
> >
> > -int run = 1;
>
> not to mention it was just code motion.
>
> > @@ -426,6 +452,9 @@ int main(int argc, char **argv)
> > return -1;
> > }
> >
> > + virConnectSetCloseCallback(dconn,
> > + connectClose, NULL, NULL);
>
> :) I was right - passing a NULL opaque is useful!
And pasing NULL is already allowed.
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list