[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 07/16] vl.c: Use "%s support disabled" error
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 07/16] vl.c: Use "%s support disabled" error messages consistently |
Date: |
Thu, 29 Oct 2015 17:59:13 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eduardo Habkost <address@hidden> writes:
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> 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?
> @@ -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);
> }
- [Qemu-devel] [PATCH v2 01/16] vl.c: Replace fprintf(stderr) with error_report(), (continued)
- [Qemu-devel] [PATCH v2 01/16] vl.c: Replace fprintf(stderr) with error_report(), Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 03/16] vl.c: Remove periods from error_report() error messages, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 02/16] vl.c: Use error_report() when reporting shutdown signal, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 04/16] vl.c: Use "warning:" prefix consistently on warnings, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 06/16] vl.c: Use 'quotes' instead of `quotes' in messages, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 05/16] vl.c: Use "cannot" instead of "can not" in error messages, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 08/16] vl.c: Simplify "ignoring deprecated option" warnings, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 07/16] vl.c: Use "%s support disabled" error messages consistently, Eduardo Habkost, 2015/10/28
- Re: [Qemu-devel] [PATCH v2 07/16] vl.c: Use "%s support disabled" error messages consistently,
Markus Armbruster <=
- [Qemu-devel] [PATCH v2 09/16] vl.c: Reword -no-kvm-pit-reinjection deprecation warning, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 10/16] vl.c: Convert error sentences to simpler phrases, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 11/16] vl.c: Remove unnecessary uppercase in error messages, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 12/16] vl.c: trivial: Don't wrap lines unnecessarily, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 14/16] vl.c: Simplify date format error message, Eduardo Habkost, 2015/10/28
- [Qemu-devel] [PATCH v2 13/16] vl.c: Reword -machine help error messages, Eduardo Habkost, 2015/10/28