[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/19] Remove double error message in qemu_optio
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH 03/19] Remove double error message in qemu_option_set() |
Date: |
Thu, 10 Sep 2009 21:03:12 +0300 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Thu, Sep 10, 2009 at 04:18:45PM +0100, Mark McLoughlin wrote:
> qemu_opt_set() prints an error message in all failure cases, so
> qemu_set_option() doesn't need to print another error.
>
> Signed-off-by: Mark McLoughlin <address@hidden>
Same comment about {} here
> ---
> qemu-config.c | 6 ++----
> 1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/qemu-config.c b/qemu-config.c
> index 61e6ed1..733cc24 100644
> --- a/qemu-config.c
> +++ b/qemu-config.c
> @@ -186,11 +186,9 @@ int qemu_set_option(const char *str)
> return -1;
> }
>
> - if (qemu_opt_set(opts, arg, str+offset+1) == -1) {
> - fprintf(stderr, "failed to set \"%s\" for %s \"%s\"\n",
> - arg, lists[i]->name, id);
> + if (qemu_opt_set(opts, arg, str+offset+1) == -1)
> return -1;
> - }
> +
> return 0;
> }
>
> --
> 1.6.2.5
>
>
- [Qemu-devel] [PATCH 11/19] Port -net tap to QemuOpts, (continued)
- [Qemu-devel] [PATCH 11/19] Port -net tap to QemuOpts, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 01/19] Suppress more more kraxelisms, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 02/19] Remove bogus error message from qemu_opts_set(), Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 07/19] Never overwrite a QemuOpt, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 04/19] Remove double error message for -device option parsing, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 03/19] Remove double error message in qemu_option_set(), Mark McLoughlin, 2009/09/10
- Re: [Qemu-devel] [PATCH 03/19] Remove double error message in qemu_option_set(),
Michael S. Tsirkin <=
- [Qemu-devel] [PATCH 08/19] Add qemu_net_opts, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 05/19] Make qemu_opts_parse() handle empty strings, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 18/19] Port PCI NIC hotplug to QemuOpts, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 12/19] Port -net socket to QemuOpts, Mark McLoughlin, 2009/09/10
- [Qemu-devel] [PATCH 06/19] Add qemu_opts_validate() for post parsing validation, Mark McLoughlin, 2009/09/10