diff options
author | Zhao Liu <zhao1.liu@intel.com> | 2024-10-29 16:59:34 +0800 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2024-12-10 18:49:25 +0100 |
commit | 281305d3e08ac7330dfe7cf7b3978c119a888bad (patch) | |
tree | 5ba562f52b60a80a672eea42ccdab1bedf5c2315 /include | |
parent | 4e40d50558432c0e2344ee1838e58e6299e63884 (diff) | |
download | qemu-281305d3e08ac7330dfe7cf7b3978c119a888bad.zip qemu-281305d3e08ac7330dfe7cf7b3978c119a888bad.tar.gz qemu-281305d3e08ac7330dfe7cf7b3978c119a888bad.tar.bz2 |
qom/object: Remove type_register()
At present, type_register() and type_register_static() are identical,
although their documentation expects the *_static variant to accept
the Typeinfo with the strings that have the static lifetime.
However, the code implementation doesn't have any check or guarantee for
static lifetime. In fact, this is unnecessary because type_new()
duplicates all strings, thereby taking ownership of them.
Therefore, type_register() and type_register_static() are redundant, so
one of them should be removed.
Since the changes required to remove type_register() were smaller,
type_register() was replaced with type_register_static() throughout the
code base. Drop its definition, and delete the requirement about string
lifetime from the documentation.
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Link: https://lore.kernel.org/r/20241029085934.2799066-17-zhao1.liu@intel.com
Diffstat (limited to 'include')
-rw-r--r-- | include/qom/object.h | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/include/qom/object.h b/include/qom/object.h index 43c1359..a201c97 100644 --- a/include/qom/object.h +++ b/include/qom/object.h @@ -880,25 +880,11 @@ const char *object_get_typename(const Object *obj); * type_register_static: * @info: The #TypeInfo of the new type. * - * @info and all of the strings it points to should exist for the life time - * that the type is registered. - * * Returns: the new #Type. */ Type type_register_static(const TypeInfo *info); /** - * type_register: - * @info: The #TypeInfo of the new type - * - * Unlike type_register_static(), this call does not require @info or its - * string members to continue to exist after the call returns. - * - * Returns: the new #Type. - */ -Type type_register(const TypeInfo *info); - -/** * type_register_static_array: * @infos: The array of the new type #TypeInfo structures. * @nr_infos: number of entries in @infos |