[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/16] qemu-option: parse_option_size(): use err
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 04/16] qemu-option: parse_option_size(): use error_set() |
Date: |
Fri, 18 May 2012 16:03:30 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.4) Gecko/20120422 Thunderbird/10.0.4 |
On 05/17/12 16:33, Luiz Capitulino wrote:
> @@ -291,8 +290,10 @@ int set_option_parameter(QEMUOptionParameter *list,
> const char *name,
> break;
>
> case OPT_SIZE:
> - if (parse_option_size(name, value, &list->value.n) == -1)
> - return -1;
> + parse_option_size(name, value, &list->value.n, &local_err);
> + if (error_is_set(&local_err)) {
> + goto exit_err;
> + }
> break;
>
> default:
> @@ -602,7 +603,8 @@ static int qemu_opt_parse(QemuOpt *opt)
> &local_err);
> break;
> case QEMU_OPT_SIZE:
> - return parse_option_size(opt->name, opt->str, &opt->value.uint);
> + parse_option_size(opt->name, opt->str, &opt->value.uint, &local_err);
> + break;
> default:
> abort();
> }
... and then the set_option_parameter() change would look like the
qemu_opt_parse() change (no if or goto). But it's fine.
Laszlo
- [Qemu-devel] [PATCH 05/16] qemu-option: qemu_opt_parse(): use error_set(), (continued)
- [Qemu-devel] [PATCH 05/16] qemu-option: qemu_opt_parse(): use error_set(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 10/16] qerror: introduce QERR_INVALID_OPTION_GROUP, Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 09/16] qemu-option: qemu_opts_from_qdict(): use error_set(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 07/16] qemu-option: opt_set(): use error_set(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 03/16] qemu-option: parse_option_bool(): use error_set(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 08/16] qemu-option: introduce qemu_opt_set_err(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 04/16] qemu-option: parse_option_size(): use error_set(), Luiz Capitulino, 2012/05/17
- Re: [Qemu-devel] [PATCH 04/16] qemu-option: parse_option_size(): use error_set(),
Laszlo Ersek <=
- [Qemu-devel] [PATCH 13/16] net: purge the monitor object from all init functions, Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 11/16] qemu-config: find_list(): use error_set(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 15/16] qapi: convert netdev_add, Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 16/16] qapi: convert netdev_del, Luiz Capitulino, 2012/05/17