[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/7] vl: Add DT_COCOA DisplayType value
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/7] vl: Add DT_COCOA DisplayType value |
Date: |
Fri, 13 Nov 2015 16:49:09 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
Am 12.11.2015 um 20:02 schrieb Eduardo Habkost:
> Instead of reusing DT_SDL for Cocoa, use DT_COCOA to indicate
> that a Cocoa display was requested.
>
> configure already ensures CONFIG_COCOA and CONFIG_SDL are never
> set at the same time. The only case where DT_SDL is used outside
> a #ifdef CONFIG_SDL block is in the no_frame/alt_grab/ctrl_grab
> check. That means the only user-visible change is that we will
> start printing a warning if the SDL-specific options are used in
> Cocoa mode. This is a bugfix, because no_frame/alt_grab/ctrl_grab
> are not used by Cocoa code.
>
> Cc: Andreas Färber <address@hidden>
> Cc: Peter Maydell <address@hidden>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> include/sysemu/sysemu.h | 1 +
> vl.c | 6 ++++--
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
> index f992494..0f4e520 100644
> --- a/include/sysemu/sysemu.h
> +++ b/include/sysemu/sysemu.h
> @@ -137,6 +137,7 @@ typedef enum DisplayType
> DT_DEFAULT,
> DT_CURSES,
> DT_SDL,
> + DT_COCOA,
> DT_GTK,
> DT_NOGRAPHIC,
> DT_NONE,
> diff --git a/vl.c b/vl.c
> index 7d993a5..bba1b0a 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4247,8 +4247,10 @@ int main(int argc, char **argv, char **envp)
> if (display_type == DT_DEFAULT && !display_remote) {
> #if defined(CONFIG_GTK)
> display_type = DT_GTK;
> -#elif defined(CONFIG_SDL) || defined(CONFIG_COCOA)
> +#elif defined(CONFIG_SDL)
> display_type = DT_SDL;
> +#elif defined(CONFIG_COCOA)
> + display_type = DT_COCOA;
> #elif defined(CONFIG_VNC)
> vnc_parse("localhost:0,to=99,id=default", &error_abort);
> show_vnc_port = 1;
> @@ -4588,7 +4590,7 @@ int main(int argc, char **argv, char **envp)
> sdl_display_init(ds, full_screen, no_frame);
> break;
> #elif defined(CONFIG_COCOA)
> - case DT_SDL:
> + case DT_COCOA:
> cocoa_display_init(ds, full_screen);
> break;
> #endif
Sounds like a good idea, but haven't reviewed those extra DT_SDL uses,
Acked-by: Andreas Färber <address@hidden>
Regards,
Andreas
- [Qemu-devel] [PATCH v2 0/7] vl: graphics stubs + #ifdef cleanup, Eduardo Habkost, 2015/11/12
- [Qemu-devel] [PATCH v2 1/7] vl: Add DT_COCOA DisplayType value, Eduardo Habkost, 2015/11/12
- Re: [Qemu-devel] [PATCH v2 1/7] vl: Add DT_COCOA DisplayType value,
Andreas Färber <=
- [Qemu-devel] [PATCH v2 2/7] stubs: VNC initialization stubs, Eduardo Habkost, 2015/11/12
- [Qemu-devel] [PATCH v2 3/7] stubs: curses_display_init() stub, Eduardo Habkost, 2015/11/12
- [Qemu-devel] [PATCH v2 4/7] stubs: SDL initialization stubs, Eduardo Habkost, 2015/11/12
- [Qemu-devel] [PATCH v2 5/7] stubs: cocoa_display_init() stub, Eduardo Habkost, 2015/11/12
- [Qemu-devel] [PATCH v2 6/7] stubs: gtk_display_init() stub, Eduardo Habkost, 2015/11/12
- [Qemu-devel] [PATCH v2 7/7] spice: Initialization stubs on qemu-spice.h, Eduardo Habkost, 2015/11/12
- Re: [Qemu-devel] [PATCH v2 0/7] vl: graphics stubs + #ifdef cleanup, Eric Blake, 2015/11/12