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
