diff options
author | Kevin Wolf <kwolf@redhat.com> | 2021-03-11 17:42:53 +0100 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2021-03-19 10:18:17 +0100 |
commit | f3b70e0779c84a5c220ca67610b27cbe672d986a (patch) | |
tree | a10f6efa94044965d1c1139d2506b49c56a3f0c5 /chardev/char.c | |
parent | 5965243641d797b2270082c5a4eab49cb81fc8f0 (diff) | |
download | qemu-f3b70e0779c84a5c220ca67610b27cbe672d986a.zip qemu-f3b70e0779c84a5c220ca67610b27cbe672d986a.tar.gz qemu-f3b70e0779c84a5c220ca67610b27cbe672d986a.tar.bz2 |
char: Simplify chardev_name_foreach()
Both callers use callbacks that don't do anything when they are called
for CLI aliases. Instead of passing the cli_alias parameter, just don't
call the callbacks for aliases in the first place.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-Id: <20210311164253.338723-4-kwolf@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'chardev/char.c')
-rw-r--r-- | chardev/char.c | 24 |
1 files changed, 5 insertions, 19 deletions
diff --git a/chardev/char.c b/chardev/char.c index 7be9579..140d6d9 100644 --- a/chardev/char.c +++ b/chardev/char.c @@ -548,7 +548,7 @@ static struct ChardevAlias { }; typedef struct ChadevClassFE { - void (*fn)(const char *name, bool is_cli_alias, void *opaque); + void (*fn)(const char *name, void *opaque); void *opaque; } ChadevClassFE; @@ -562,33 +562,23 @@ chardev_class_foreach(ObjectClass *klass, void *opaque) return; } - fe->fn(object_class_get_name(klass) + 8, false, fe->opaque); + fe->fn(object_class_get_name(klass) + 8, fe->opaque); } static void -chardev_name_foreach(void (*fn)(const char *name, bool is_cli_alias, - void *opaque), +chardev_name_foreach(void (*fn)(const char *name, void *opaque), void *opaque) { ChadevClassFE fe = { .fn = fn, .opaque = opaque }; - int i; object_class_foreach(chardev_class_foreach, TYPE_CHARDEV, false, &fe); - - for (i = 0; i < (int)ARRAY_SIZE(chardev_alias_table); i++) { - fn(chardev_alias_table[i].alias, true, opaque); - } } static void -help_string_append(const char *name, bool is_cli_alias, void *opaque) +help_string_append(const char *name, void *opaque) { GString *str = opaque; - if (is_cli_alias) { - return; - } - g_string_append_printf(str, "\n %s", name); } @@ -810,15 +800,11 @@ ChardevInfoList *qmp_query_chardev(Error **errp) } static void -qmp_prepend_backend(const char *name, bool is_cli_alias, void *opaque) +qmp_prepend_backend(const char *name, void *opaque) { ChardevBackendInfoList **list = opaque; ChardevBackendInfo *value; - if (is_cli_alias) { - return; - } - value = g_new0(ChardevBackendInfo, 1); value->name = g_strdup(name); QAPI_LIST_PREPEND(*list, value); |