On Thu, Oct 29, 2015 at 05:59:13PM +0100, Markus Armbruster wrote:
> Eduardo Habkost <[email protected]> writes:
>
> > Signed-off-by: Eduardo Habkost <[email protected]>
> > ---
> > vl.c | 34 ++++++++++++++++------------------
> > 1 file changed, 16 insertions(+), 18 deletions(-)
> >
> > diff --git a/vl.c b/vl.c
> > index d417dd9..f37e3a9 100644
> > --- a/vl.c
> > +++ b/vl.c
> > @@ -1018,8 +1018,7 @@ static int parse_sandbox(void *opaque, QemuOpts
> > *opts, Error **errp)
> > return -1;
> > }
> > #else
> > - error_report("sandboxing request but seccomp is not compiled "
> > - "into this build");
> > + error_report("seccomp support disabled");
> > return -1;
> > #endif
> > }
>
> Yes, please.
>
> > @@ -2112,7 +2111,7 @@ static DisplayType select_display(const char *p)
> > opts = nextopt;
> > }
> > #else
> > - error_report("SDL support is disabled");
> > + error_report("SDL support disabled");
> > exit(1);
> > #endif
> > } else if (strstart(p, "vnc", &opts)) {
>
> I'd prefer the old wording. Drew?
I like the 'is' version better too. I'm OK with both though.
>
> > @@ -2128,14 +2127,14 @@ static DisplayType select_display(const char *p)
> > exit(1);
> > }
> > #else
> > - error_report("VNC support is disabled");
> > + error_report("VNC support disabled");
> > exit(1);
> > #endif
> > } else if (strstart(p, "curses", &opts)) {
> > #ifdef CONFIG_CURSES
> > display = DT_CURSES;
> > #else
> > - error_report("Curses support is disabled");
> > + error_report("curses support disabled");
> > exit(1);
> > #endif
> > } else if (strstart(p, "gtk", &opts)) {
> > @@ -2170,7 +2169,7 @@ static DisplayType select_display(const char *p)
> > opts = nextopt;
> > }
> > #else
> > - error_report("GTK support is disabled");
> > + error_report("GTK support disabled");
> > exit(1);
> > #endif
> > } else if (strstart(p, "none", &opts)) {
> > @@ -2635,7 +2634,7 @@ static gint machine_class_cmp(gconstpointer a,
> > gconstpointer b)
> > return mc;
> > }
> > if (name && !is_help_option(name)) {
> > - error_report("Unsupported machine type");
> > + error_report("unsupported machine type");
> > error_printf("Use -machine help to list supported machines!\n");
> > } else {
> > printf("Supported machines are:\n");
>
> This belongs to PATCH 11.
>
> > @@ -3011,7 +3010,7 @@ int main(int argc, char **argv, char **envp)
> > runstate_init();
> >
> > if (qcrypto_init(&err) < 0) {
> > - error_report("Cannot initialize crypto: %s",
> > error_get_pretty(err));
> > + error_report("cannot initialize crypto: %s",
> > error_get_pretty(err));
> > exit(1);
> > }
> > rtc_clock = QEMU_CLOCK_HOST;
>
> Likewise.
>
> > @@ -3212,7 +3211,7 @@ int main(int argc, char **argv, char **envp)
> > #ifdef CONFIG_CURSES
> > display_type = DT_CURSES;
> > #else
> > - error_report("Curses support is disabled");
> > + error_report("curses support disabled");
> > exit(1);
> > #endif
> > break;
>
> Likewise.
>
> > @@ -3440,7 +3439,7 @@ int main(int argc, char **argv, char **envp)
> > case QEMU_OPTION_fsdev:
> > olist = qemu_find_opts("fsdev");
> > if (!olist) {
> > - error_report("fsdev is not supported by this qemu
> > build");
> > + error_report("fsdev support disabled");
> > exit(1);
> > }
> > opts = qemu_opts_parse_noisily(olist, optarg, true);
> > @@ -3455,7 +3454,7 @@ int main(int argc, char **argv, char **envp)
> >
> > olist = qemu_find_opts("virtfs");
> > if (!olist) {
> > - error_report("virtfs is not supported by this qemu
> > build");
> > + error_report("virtfs support disabled");
> > exit(1);
> > }
> > opts = qemu_opts_parse_noisily(olist, optarg, true);
> > @@ -3593,7 +3592,7 @@ int main(int argc, char **argv, char **envp)
> > display_type = DT_SDL;
> > break;
> > #else
> > - error_report("SDL support is disabled");
> > + error_report("SDL support disabled");
> > exit(1);
> > #endif
> > case QEMU_OPTION_pidfile:
> > @@ -3705,7 +3704,7 @@ int main(int argc, char **argv, char **envp)
> > exit(1);
> > }
> > #else
> > - error_report("VNC support is disabled");
> > + error_report("VNC support disabled");
> > exit(1);
> > #endif
> > break;
> > @@ -3899,7 +3898,7 @@ int main(int argc, char **argv, char **envp)
> > case QEMU_OPTION_spice:
> > olist = qemu_find_opts("spice");
> > if (!olist) {
> > - error_report("spice is not supported by this qemu
> > build");
> > + error_report("spice support disabled");
> > exit(1);
> > }
> > opts = qemu_opts_parse_noisily(olist, optarg, false);
> > @@ -3947,8 +3946,7 @@ int main(int argc, char **argv, char **envp)
> > exit(1);
> > }
> > #else
> > - error_report("File descriptor passing is disabled on this "
> > - "platform");
> > + error_report("fd passing disabled on this platform");
> > exit(1);
> > #endif
> > break;
>
> It's stick to "file descriptor".
>
> > @@ -4249,9 +4247,9 @@ int main(int argc, char **argv, char **envp)
> > #endif
> > if (request_opengl == 1 && display_opengl == 0) {
> > #if defined(CONFIG_OPENGL)
> > - error_report("OpenGL is not supported by the display");
> > + error_report("OpenGL not supported by the display");
> > #else
> > - error_report("QEMU was built without opengl support");
> > + error_report("OpenGL support disabled");
> > #endif
> > exit(1);
> > }
>
