[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/16] qapi: Use qapi_enum_parse() in input_type
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 10/16] qapi: Use qapi_enum_parse() in input_type_enum() |
Date: |
Thu, 24 Aug 2017 13:30:28 +0200 |
On Thu, Aug 24, 2017 at 10:46 AM, Markus Armbruster <address@hidden> wrote:
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
> ---
> qapi/qapi-visit-core.c | 13 ++++---------
> 1 file changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/qapi/qapi-visit-core.c b/qapi/qapi-visit-core.c
> index ed6d2af..ec83ff6 100644
> --- a/qapi/qapi-visit-core.c
> +++ b/qapi/qapi-visit-core.c
> @@ -14,6 +14,7 @@
>
> #include "qemu/osdep.h"
> #include "qapi/error.h"
> +#include "qapi/util.h"
> #include "qemu-common.h"
> #include "qapi/qmp/qobject.h"
> #include "qapi/qmp/qerror.h"
> @@ -353,7 +354,7 @@ static void input_type_enum(Visitor *v, const char *name,
> int *obj,
> const char *const strings[], Error **errp)
> {
> Error *local_err = NULL;
> - int64_t value = 0;
> + int64_t value;
> char *enum_str;
>
> visit_type_str(v, name, &enum_str, &local_err);
> @@ -362,14 +363,8 @@ static void input_type_enum(Visitor *v, const char
> *name, int *obj,
> return;
> }
>
> - while (strings[value] != NULL) {
> - if (strcmp(strings[value], enum_str) == 0) {
> - break;
> - }
> - value++;
> - }
> -
> - if (strings[value] == NULL) {
> + value = qapi_enum_parse(strings, enum_str, -1, NULL);
> + if (value < 0) {
> error_setg(errp, QERR_INVALID_PARAMETER, enum_str);
> g_free(enum_str);
> return;
> --
> 2.7.5
>
>
--
Marc-André Lureau
- [Qemu-devel] [PATCH 00/16] qapi: Rework mapping of enum value to string, Markus Armbruster, 2017/08/24
- [Qemu-devel] [PATCH 10/16] qapi: Use qapi_enum_parse() in input_type_enum(), Markus Armbruster, 2017/08/24
- Re: [Qemu-devel] [PATCH 10/16] qapi: Use qapi_enum_parse() in input_type_enum(),
Marc-André Lureau <=
- [Qemu-devel] [PATCH 14/16] qapi: Convert indirect uses of FOO_lookup[...] to qapi_enum_lookup(), Markus Armbruster, 2017/08/24
- [Qemu-devel] [PATCH 12/16] qapi: Generate FOO_str() macro for QAPI enum FOO, Markus Armbruster, 2017/08/24
- [Qemu-devel] [PATCH 08/16] quorum: Use qapi_enum_parse() in quorum_open(), Markus Armbruster, 2017/08/24
- [Qemu-devel] [PATCH 16/16] qapi: drop the sentinel in enum array, Markus Armbruster, 2017/08/24
- [Qemu-devel] [PATCH 11/16] qapi: Avoid unnecessary use of enum lookup table's sentinel, Markus Armbruster, 2017/08/24
- [Qemu-devel] [PATCH 04/16] tpm: Clean up model registration & lookup, Markus Armbruster, 2017/08/24