[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/16] qemu-option: parse_option_bool(): use err
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 03/16] qemu-option: parse_option_bool(): use error_set() |
Date: |
Fri, 18 May 2012 15:59:36 +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:
> Note that set_option_parameter() callers still expect automatic error
> reporting with QError, so set_option_parameter() calls
> qerror_report_err() to keep the same semantics.
>
> Signed-off-by: Luiz Capitulino <address@hidden>
> ---
> qemu-option.c | 22 +++++++++++++++-------
> 1 file changed, 15 insertions(+), 7 deletions(-)
>
> diff --git a/qemu-option.c b/qemu-option.c
> index 72dcb56..a8b50af 100644
> --- a/qemu-option.c
> +++ b/qemu-option.c
> @@ -169,7 +169,8 @@ QEMUOptionParameter
> *get_option_parameter(QEMUOptionParameter *list,
> return NULL;
> }
>
> -static int parse_option_bool(const char *name, const char *value, bool *ret)
> +static void parse_option_bool(const char *name, const char *value, bool *ret,
> + Error **errp)
> {
> if (value != NULL) {
> if (!strcmp(value, "on")) {
> @@ -177,13 +178,11 @@ static int parse_option_bool(const char *name, const
> char *value, bool *ret)
> } else if (!strcmp(value, "off")) {
> *ret = 0;
> } else {
> - qerror_report(QERR_INVALID_PARAMETER_VALUE, name, "'on' or
> 'off'");
> - return -1;
> + error_set(errp,QERR_INVALID_PARAMETER_VALUE, name, "'on' or
> 'off'");
> }
> } else {
> *ret = 1;
> }
> - return 0;
> }
>
> static void parse_option_number(const char *name, const char *value,
> @@ -263,6 +262,7 @@ int set_option_parameter(QEMUOptionParameter *list, const
> char *name,
> const char *value)
> {
> bool flag;
> + Error *local_err = NULL;
>
> // Find a matching parameter
> list = get_option_parameter(list, name);
> @@ -274,8 +274,10 @@ int set_option_parameter(QEMUOptionParameter *list,
> const char *name,
> // Process parameter
> switch (list->type) {
> case OPT_FLAG:
> - if (parse_option_bool(name, value, &flag) == -1)
> - return -1;
> + parse_option_bool(name, value, &flag, &local_err);
> + if (error_is_set(&local_err)) {
> + goto exit_err;
> + }
> list->value.n = flag;
> break;
>
> @@ -299,6 +301,11 @@ int set_option_parameter(QEMUOptionParameter *list,
> const char *name,
> }
>
> return 0;
> +
> +exit_err:
> + qerror_report_err(local_err);
> + error_free(local_err);
> + return -1;
> }
I think this may have been simpler (= goto-less) like this:
case OPT_FLAG:
parse_option_bool(name, value, &flag, &local_err);
if (!error_is_set(&local_err)) {
list->value.n = flag;
}
break;
and in the end, before the "return 0", another check with error_is_set().
But it appears correct this way too, of course.
Laszlo
- [Qemu-devel] [PATCH qmp-next v4 00/16]: qapi: convert netdev_add & netdev_del, Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 02/16] qemu-option: parse_option_number(): use error_set(), Luiz Capitulino, 2012/05/17
- [Qemu-devel] [PATCH 01/16] qemu-option: qemu_opts_create(): use error_set(), Luiz Capitulino, 2012/05/17
- [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
- Re: [Qemu-devel] [PATCH 03/16] qemu-option: parse_option_bool(): use error_set(),
Laszlo Ersek <=
- [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
- [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