[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] spice: drop incorrect vm_change_state_handler()
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH] spice: drop incorrect vm_change_state_handler() opaque |
Date: |
Wed, 2 Jan 2013 16:56:58 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Dec 19, 2012 at 02:07:16PM +0100, Stefan Hajnoczi wrote:
> The spice_server pointer is a global variable and
> vm_change_state_handler() therefore does not use its opaque parameter.
>
> The vm change state handler is added with a pointer to the spice_server
> pointer. This is useless and we probably would not want 2 levels of
> pointers.
>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> ui/spice-core.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/ui/spice-core.c b/ui/spice-core.c
> index ac46deb..c128c0b 100644
> --- a/ui/spice-core.c
> +++ b/ui/spice-core.c
> @@ -709,7 +709,7 @@ void qemu_spice_init(void)
> qemu_spice_input_init();
> qemu_spice_audio_init();
>
> - qemu_add_vm_change_state_handler(vm_change_state_handler, &spice_server);
> + qemu_add_vm_change_state_handler(vm_change_state_handler, NULL);
>
> g_free(x509_key_file);
> g_free(x509_cert_file);
> @@ -736,8 +736,7 @@ int qemu_spice_add_interface(SpiceBaseInstance *sin)
> */
> spice_server = spice_server_new();
> spice_server_init(spice_server, &core_interface);
> - qemu_add_vm_change_state_handler(vm_change_state_handler,
> - &spice_server);
> + qemu_add_vm_change_state_handler(vm_change_state_handler, NULL);
> }
>
> return spice_server_add_interface(spice_server, sin);
> --
> 1.8.0.2
Gerd, would you like to take this through the spice queue or should I
put it in trivial-patches?
Stefan
- Re: [Qemu-devel] [PATCH] spice: drop incorrect vm_change_state_handler() opaque,
Stefan Hajnoczi <=