On Thu, Oct 29, 2015 at 05:59:13PM +0100, Markus Armbruster wrote: > Eduardo Habkost <ehabk...@redhat.com> writes: > > > Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> > > --- > > 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); > > } >