qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH trivial] vl: always define no_frame


From: Peter Wu
Subject: Re: [Qemu-devel] [PATCH trivial] vl: always define no_frame
Date: Sat, 15 Jun 2013 12:51:37 +0200
User-agent: KMail/4.10.4 (Linux/3.9.1-1-custom; KDE/4.10.4; x86_64; ; )

On Saturday 15 June 2013 14:42:03 Michael Tokarev wrote:

> Commit 047d4e151dd46 "Unbreak -no-quit for GTK, validate SDL options" broke

> build of qemu without sdl, by referencing `no_frame' variable which is

> defined inside #if SDL block. Fix that by defining that variable

> unconditionally.

>

> This is a better fix for the build issue introduced by that patch than

> a revert. This change keeps the new functinality introduced by that patch

> and just fixes the compilation. It still is not a complete fix around the

> original issue (not working -no-frame et al with -display gtk), because it

> makes only the legacy interface working, not the new suboption interface,

> so a few more changes are needed.

 

Reviewed-by: Peter Wu <address@hidden>

 

-no-frame was deemed to be a SDL feature that was unnecessary for GTK, hence dropped. At run-time, it is checked whether -no-frame makes sense or not (that is, if `-display sdl` was specified or not). Original patch and discussion can be found at http://lists.nongnu.org/archive/html/qemu-devel/2013-06/msg01297.html.

 

As for the discussion about legacy options[1], are there plans to remove `-sdl` (replaced by `-display sdl`) and move the `-*-grab` and `-no-frame` to `-display sdl` options? If that is the idea, what about printing a message that the mentioned options are deprecated and removed in a future version?

 

Regards,

Peter

 

[1]: http://lists.nongnu.org/archive/html/qemu-devel/2013-06/msg02437.html

 

> Cc: Peter Wu <address@hidden>

> Cc: address@hidden

> Signed-off-by: Michael Tokarev <address@hidden>

> ---

> vl.c | 2 --

> 1 file changed, 2 deletions(-)

>

> diff --git a/vl.c b/vl.c

> index 9f8fd6e..f94ec9c 100644

> --- a/vl.c

> +++ b/vl.c

> @@ -199,9 +199,7 @@ static int rtc_date_offset = -1; /* -1 means no change

> */ QEMUClock *rtc_clock;

> int vga_interface_type = VGA_NONE;

> static int full_screen = 0;

> -#ifdef CONFIG_SDL

> static int no_frame = 0;

> -#endif

> int no_quit = 0;

> CharDriverState *serial_hds[MAX_SERIAL_PORTS];

> CharDriverState *parallel_hds[MAX_PARALLEL_PORTS];


reply via email to

[Prev in Thread] Current Thread [Next in Thread]