On Wed, Apr 20, 2022 at 09:08:02PM +0200, Victor Toso wrote:
@@ -4815,6 +4821,9 @@ typedef void (*virConnectDomainEventGenericCallback)(virConnectPtr conn, * * The callback signature to use when registering for an event of type * VIR_DOMAIN_EVENT_ID_RTC_CHANGE with virConnectDomainEventRegisterAny() + * + * Since: v0.8.0 + * */ typedef void (*virConnectDomainEventRTCChangeCallback)(virConnectPtr conn, virDomainPtr dom, @@ -4853,6 +4862,9 @@ typedef enum { * The callback signature to use when registering for an event of type * VIR_DOMAIN_EVENT_ID_WATCHDOG with virConnectDomainEventRegisterAny() * + * Since: v0.8.0 + * + * */ typedef void (*virConnectDomainEventWatchdogCallback)(virConnectPtr conn, virDomainPtr dom,
I see that Peter is going really hard with these reviews, so while I intended to look at the series before the end of the week he's probably going to be done with it way before then so I better speak up now :) The vertical whitespace is inconsistent: two different styles can be seen here, and I've also spotted instances of * ... * * Since: v... */ elsewhere. My own preference would be to adopt this last style everywhere, but I would be okay with having a single empty line after version information. Either way, it needs to be applied consistently. -- Andrea Bolognani / Red Hat / Virtualization