Re: [libvirt] [PATCH 1/2] event: fix doc typos, and doc more public methods

On 01/02/2014 09:32 AM, Eric Blake wrote: And squashing in this use of bool (hmm, once again pasting in causes line wrapping :( ) diff --git c/src/conf/object_event.c w/src/conf/object_event.c index eca6a10..bbf1972 100644 --- c/src/conf/object_event.c +++ w/src/conf/object_event.c @@ -589,33 +589,29 @@ virObjectEventQueuePush(virObjectEventQueuePtr evtQueue, } -static int +static bool virObjectEventDispatchMatchCallback(virObjectEventPtr event, virObjectEventCallbackPtr cb) { if (!cb) - return 0; + return false; if (cb->deleted) - return 0; + return false; if (!virObjectIsClass(event, cb->klass)) - return 0; + return false; if (cb->eventID != virObjectEventGetEventID(event)) - return 0; + return false; if (cb->meta) { /* Deliberately ignoring 'id' for matching, since that * will cause problems when a domain switches between * running & shutoff states & ignoring 'name' since * Xen sometimes renames guests during migration, thus - * leaving 'uuid' as the only truly reliable ID we can use*/ - - if (memcmp(event->meta.uuid, cb->meta->uuid, VIR_UUID_BUFLEN) == 0) - return 1; + * leaving 'uuid' as the only truly reliable ID we can use. */ - return 0; - } else { - return 1; + return memcmp(event->meta.uuid, cb->meta->uuid, VIR_UUID_BUFLEN) == 0; } + return true; } -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org