[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] Reorganize help output of '-display' option
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] Reorganize help output of '-display' option |
Date: |
Mon, 25 Jul 2016 10:50:23 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Robert Ho <address@hidden> writes:
> The '-display' help information is not very correct. This patch sort
> it a little.
> Also, in its help information, reveals what implicit display option
> will be chosen if no definition.
>
> P.S.
> Hi Markus,
>
> Sorry for late of sending out this patch. Really busy recently, and it
> also took me some time to understand the framework help message generation.
No problem :)
> Signed-off-by: Robert Ho <address@hidden>
> ---
> qemu-options.hx | 32 +++++++++++++++++++++++++-------
> 1 file changed, 25 insertions(+), 7 deletions(-)
>
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 17f15ad..69cf668 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -930,10 +930,26 @@ ETEXI
>
> DEF("display", HAS_ARG, QEMU_OPTION_display,
> "-display sdl[,frame=on|off][,alt_grab=on|off][,ctrl_grab=on|off]\n"
> - " [,window_close=on|off]|curses|none|\n"
> - " gtk[,grab_on_hover=on|off]|\n"
> - " vnc=<display>[,<optargs>]\n"
> - " select display type\n", QEMU_ARCH_ALL)
> + " [,window_close=on|off][,gl=on|off]|curses|none|\n"
> + "-display gtk[,grab_on_hover=on|off][,gl=on|off]|\n"
> + "-display vnc=<display>[,<optargs>]\n"
> + "-display curses\n"
> + "-display none"
> + " select display type\n"
> + "If no -display option or its sugars (sd, curses, vnc, etc) defined,"
> + " qemu will implicitly use\n"
Results in a long line of output. You misspelled sdl.
Perhaps simply:
"The default display is equivalent to\n"
> +#if defined(CONFIG_GTK)
> + "\t\"-dispaly gtk\"\n"
s/dispaly/display/
> +#elif defined(CONFIG_SDL)
> + "\t\"-display sdl\"\n"
> +#elif defined(CONFIG_COCOA)
> + "\t\"-display cocoa\"\n"
> +#elif defined(CONFIG_VNC)
> + "\t\"-vnc localhost:0,to=99,id=default\"\n"
> +#else
> + "\t\"-display none\"\n"
> +#endif
> + , QEMU_ARCH_ALL)
> STEXI
> @item -display @var{type}
> @findex -display
> @@ -980,7 +996,8 @@ the console and monitor.
> ETEXI
>
> DEF("curses", 0, QEMU_OPTION_curses,
> - "-curses use a curses/ncurses interface instead of SDL\n",
> + "-curses use a curses/ncurses interface instead of SDL; "
> + "is sugar for -display curses\n",
Long line of output. Preexisting: the "instead of SDL" part is wrong
unless CONFIG_SDL.
What about:
"-curses shorthand for -display curses\n",
> QEMU_ARCH_ALL)
> STEXI
> @item -curses
> @@ -1030,7 +1047,7 @@ Disable SDL window close capability.
> ETEXI
>
> DEF("sdl", 0, QEMU_OPTION_sdl,
> - "-sdl enable SDL\n", QEMU_ARCH_ALL)
> + "-sdl enable SDL; is sugar for -display sdl\n", QEMU_ARCH_ALL)
If we change -curses as suggested, then this should become
"-sdl enable SDL; is sugar for -display sdl\n",
for consistency.
> STEXI
> @item -sdl
> @findex -sdl
> @@ -1227,7 +1244,8 @@ Set the initial graphical resolution and depth (PPC,
> SPARC only).
> ETEXI
>
> DEF("vnc", HAS_ARG, QEMU_OPTION_vnc ,
> - "-vnc display start a VNC server on display\n", QEMU_ARCH_ALL)
> + "-vnc display start a VNC server on display; is sugar for"
> +" -display vnc=display\n", QEMU_ARCH_ALL)
Long line of output.
If we change -curses as suggested, then this should become
-vnc display start a VNC server on display; is sugar for -display
vnc=display
"-vnc <display> shorthand for -display vnc=<display>\n",
for consistency.
> STEXI
> @item -vnc @var{display}[,@var{option}[,@var{option}[,...]]]
> @findex -vnc
Thanks for tackling this!