Re: [libvirt] [PATCH 0/2] event cleanups

Apologies for not creating a cover letter originally, so now this thread is all awkward. While I still have pending event patches (for my new proposed API, among other things), those can be in a new series. The patches in this series represent cleanups to a number of bugs that I found while trying to prepare my further patches. I ended up enhancing the testsuite to catch quite a few issues, and left tests as separate commits. I rebased things so that as presented in this series, most tests are presented first (to expose the failure) then the patch to fix the issue; but my plans are to squash them together before actually pushing so that 'make check' works for all pushed commits. (The only exception to the naming convention is that patch 4/2 is the test for 2/2). The final series will look like: Eric Blake (10): event: use bool in more places event: make deregister return value match docs event: use newer array management macros event: share state driver between test:///default connections event: rename confusing variable in test, remote drivers event: don't let old-style events clobber per-domain events event: tighten scope of object_event event: properly filter count of remaining events event: don't allow mix of old- and new-style registration event: don't turn offline domain into global event src/conf/domain_event.c | 185 +++----------------- src/conf/network_event.c | 13 +- src/conf/object_event.c | 363 +++++++++++++++++++++++++--------------- src/conf/object_event.h | 2 - src/conf/object_event_private.h | 61 +------ src/libvirt.c | 17 +- src/libxl/libxl_driver.c | 35 ++-- src/lxc/lxc_driver.c | 32 ++-- src/network/bridge_driver.c | 11 +- src/remote/remote_driver.c | 39 ++--- src/test/test_driver.c | 132 +++++++++------ src/uml/uml_driver.c | 29 ++-- src/vbox/vbox_tmpl.c | 32 ++-- src/xen/xen_driver.c | 27 +-- tests/objecteventtest.c | 192 +++++++++++++++++++-- 15 files changed, 639 insertions(+), 531 deletions(-) -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org