[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 16/26] qapi: add conditions to VNC type/commands
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH 16/26] qapi: add conditions to VNC type/commands/events on the schema |
Date: |
Wed, 23 Aug 2017 17:07:12 +0200 |
On Thu, 2017-08-17 at 10:56 +0200, Markus Armbruster wrote:
> Gerd, can we delete the vnc_init_func() stub? Things still compile
> for
> me when I do.
>
> diff --git a/include/ui/console.h b/include/ui/console.h
> index 7262bef..2c3b2cd 100644
> --- a/include/ui/console.h
> +++ b/include/ui/console.h
> @@ -484,11 +484,6 @@ static inline QemuOpts *vnc_parse(const char
> *str, Error **errp)
> error_setg(errp, "VNC support is disabled");
> return NULL;
> }
> -static inline int vnc_init_func(void *opaque, QemuOpts *opts, Error
> **errp)
> -{
> - error_setg(errp, "VNC support is disabled");
> - return -1;
> -}
> #endif
Looking at a663fbd9e2f65fae81018d81f231ad79510cf9fb. Guess we want fix
qemu_opts_foreach to not barf on NULL, then revert that commit, so
f8c75b2486 (adding Eduardo to Cc) works as intended.
Beside that: The inline for vnc_init_func looks pointless (it's used
via function pointer). Also it can be empty as it will never be
actually called. Which is fine, vnc_parse will throw an error if
someone tries to use vnc with a binary without vnc support.
cheers,
Gerd