Re: [PATCH 02/33] qemu: monitor: Remove return value from qemuMonitorEmit* functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a %A in %Y, Peter Krempa wrote:
> The callers in the monitor code invoking the callbacks after events are
> received don't actually check the return value from the callbacks and
> there isn't really anything we could do on failure.
> 
> Remove the return value from the intermediary functions so we can later
> remove them from the callback prototypes.
> 
> Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
> ---
>  src/qemu/qemu_monitor.c | 204 ++++++++++++----------------------------
>  src/qemu/qemu_monitor.h | 176 +++++++++++++++++-----------------
>  2 files changed, 150 insertions(+), 230 deletions(-)
> 

> diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
> index 265c25c521..2fa5151652 100644
> --- a/src/qemu/qemu_monitor.h
> +++ b/src/qemu/qemu_monitor.h
> -int qemuMonitorEmitTrayChange(qemuMonitor *mon,
> -                              const char *devAlias,
> -                              const char *devid,
> -                              int reason);
> -int qemuMonitorEmitPMWakeup(qemuMonitor *mon);
> -int qemuMonitorEmitPMSuspend(qemuMonitor *mon);
> -int qemuMonitorEmitBlockJob(qemuMonitor *mon,
> +    void qemuMonitorEmitEvent(qemuMonitor *mon, const char *event,
> +                              long long seconds, unsigned int micros,
> +                              const char *details);

Indentation is off here.

Jano

> +void qemuMonitorEmitShutdown(qemuMonitor *mon, virTristateBool guest);
> +void qemuMonitorEmitReset(qemuMonitor *mon);
> +void qemuMonitorEmitStop(qemuMonitor *mon);
> +void qemuMonitorEmitResume(qemuMonitor *mon);
> +void qemuMonitorEmitRTCChange(qemuMonitor *mon, long long offset);
> +void qemuMonitorEmitWatchdog(qemuMonitor *mon, int action);




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux