On Tue, Dec 20, 2022 at 10:06:43AM +0100, Markus Armbruster wrote:
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
>  include/monitor/hmp.h | 1 +
>  include/ui/console.h  | 2 +-
>  monitor/misc.c        | 1 -
>  ui/input.c            | 5 +----
>  ui/ui-hmp-cmds.c      | 8 ++++++++
>  5 files changed, 11 insertions(+), 6 deletions(-)

>  
> -void hmp_mouse_set(Monitor *mon, const QDict *qdict)
> +void qemu_mouse_set(int index, Error **err)

This is adding a Error parameter, nit s/err/errp/

>  {
>      QemuInputHandlerState *s;
> -    int index = qdict_get_int(qdict, "index");
>      int found = 0;
>  
>      QTAILQ_FOREACH(s, &handlers, node) {

But not changing either error_report() call to error_setg(),
so the errp is unused.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|


Reply via email to