diff --git a/src/entt/meta/factory.hpp b/src/entt/meta/factory.hpp index 46744b9be..eb664ca23 100644 --- a/src/entt/meta/factory.hpp +++ b/src/entt/meta/factory.hpp @@ -21,6 +21,66 @@ namespace entt { +/** + * @cond TURN_OFF_DOXYGEN + * Internal details not to be documented. + */ + +namespace internal { + +inline internal::meta_base_node &meta_extend(internal::meta_type_node &owner, const id_type id, internal::meta_base_node &node) { + return owner.details->base.insert_or_assign(id, std::move(node)).first->second; +} + +inline internal::meta_conv_node &meta_extend(internal::meta_type_node &owner, const id_type id, internal::meta_conv_node &node) { + return owner.details->conv.insert_or_assign(id, std::move(node)).first->second; +} + +inline internal::meta_ctor_node &meta_extend(internal::meta_type_node &owner, const id_type id, internal::meta_ctor_node &node) { + return owner.details->ctor.insert_or_assign(id, std::move(node)).first->second; +} + +inline internal::meta_dtor_node &meta_extend(internal::meta_type_node &owner, internal::meta_dtor_node &node) { + return (owner.dtor = std::move(node)); +} + +inline internal::meta_data_node &meta_extend(internal::meta_type_node &owner, const id_type id, internal::meta_data_node &node) { + return owner.details->data.insert_or_assign(id, std::move(node)).first->second; +} + +inline internal::meta_func_node &meta_extend(internal::meta_type_node &owner, const id_type id, internal::meta_func_node &node) { + if(auto it = owner.details->func.find(id); it != owner.details->func.end()) { + for(auto *curr = &it->second; curr; curr = curr->next.get()) { + if(curr->invoke == node.invoke) { + node.next = std::move(curr->next); + *curr = std::move(node); + return *curr; + } + } + + // locally overloaded function + node.next = std::make_shared(std::move(owner.details->func[id])); + } + + return owner.details->func.insert_or_assign(id, std::move(node)).first->second; +} + +template +inline internal::meta_prop_node &meta_extend(Owner &owner, const id_type id, internal::meta_prop_node &node) { + if(!owner.details) { + owner.details = std::make_shared(); + } + + return (owner.details->prop[id] = std::move(node)); +} + +} // namespace internal + +/** + * Internal details not to be documented. + * @endcond + */ + /** * @brief Meta factory to be used for reflection purposes. * @@ -45,7 +105,7 @@ public: * @param target The node for which to store properties, if any. */ meta_factory(Node &target) noexcept - : node{&target} {} + : parent{&target} {} /** * @brief Assigns a property to the last meta object created. @@ -53,29 +113,30 @@ public: * Both the key and the value (if any) must be at least copy constructible. * * @tparam Value Optional type of the property value. - * @param key Property key. + * @param id Property key. * @param value Optional property value. * @return An extended meta factory for the given type. */ template - meta_factory prop(id_type key, [[maybe_unused]] Value &&...value) { - if(!node->details) { - node->details = std::make_shareddetails)::element_type>(); - } - + meta_factory prop(id_type id, [[maybe_unused]] Value &&...value) { if constexpr(sizeof...(Value) == 0u) { - node->details->prop[key] = internal::meta_prop_node{&internal::resolve}; + internal::meta_prop_node node{ + &internal::resolve}; + + internal::meta_extend(*parent, id, std::move(node)); } else { - node->details->prop[key] = internal::meta_prop_node{ + internal::meta_prop_node node{ &internal::resolve>..., std::make_shared>(std::forward(value))...}; + + internal::meta_extend(*parent, id, std::move(node)); } return *this; } private: - Node *node; + Node *parent; }; /** @@ -103,8 +164,8 @@ class meta_factory { +[](meta_handle instance, meta_any value) -> bool { return (meta_setter>(*instance.operator->(), value.as_ref()) || ...); }, &meta_getter}; - auto it = owner().details->data.insert_or_assign(id, std::move(node)).first; - return meta_factory{it->second}; + auto &&elem = internal::meta_extend(owner(), id, std::move(node)); + return meta_factory{elem}; } public: @@ -139,7 +200,7 @@ public: auto base() noexcept { static_assert(!std::is_same_v && std::is_base_of_v, "Invalid base type"); - owner().details->base[type_id().hash()] = internal::meta_base_node{ + internal::meta_base_node node{ &internal::resolve, +[](meta_any other) noexcept -> meta_any { if(auto *ptr = other.data(); ptr) { @@ -149,6 +210,7 @@ public: return forward_as_meta(*static_cast(static_cast(std::as_const(other).data()))); }}; + internal::meta_extend(owner(), type_id().hash(), std::move(node)); return *this; } @@ -168,11 +230,12 @@ public: auto conv() noexcept { using conv_type = std::remove_cv_t>>; - owner().details->conv[type_id().hash()] = internal::meta_conv_node{ + internal::meta_conv_node node{ +[](const meta_any &instance) -> meta_any { return forward_as_meta(std::invoke(Candidate, *static_cast(instance.data()))); }}; + internal::meta_extend(owner(), type_id().hash(), std::move(node)); return *this; } @@ -189,11 +252,12 @@ public: auto conv() noexcept { using conv_type = std::remove_cv_t>; - owner().details->conv[type_id().hash()] = internal::meta_conv_node{ + internal::meta_conv_node node{ +[](const meta_any &instance) -> meta_any { return forward_as_meta(static_cast(*static_cast(instance.data()))); }}; + internal::meta_extend(owner(), type_id().hash(), std::move(node)); return *this; } @@ -216,11 +280,12 @@ public: static_assert(Policy::template value, "Invalid return type for the given policy"); static_assert(std::is_same_v>, Type>, "The function doesn't return an object of the required type"); - owner().details->ctor[type_id().hash()] = internal::meta_ctor_node{ + internal::meta_ctor_node node{ descriptor::args_type::size, &meta_arg, &meta_construct}; + internal::meta_extend(owner(), type_id().hash(), std::move(node)); return *this; } @@ -238,11 +303,12 @@ public: auto ctor() noexcept { using descriptor = meta_function_helper_t; - owner().details->ctor[type_id().hash()] = internal::meta_ctor_node{ + internal::meta_ctor_node node{ descriptor::args_type::size, &meta_arg, &meta_construct}; + internal::meta_extend(owner(), type_id().hash(), std::move(node)); return *this; } @@ -267,7 +333,8 @@ public: template auto dtor() noexcept { static_assert(std::is_invocable_v, "The function doesn't accept an object of the type provided"); - owner().dtor.dtor = +[](void *instance) { std::invoke(Func, *static_cast(instance)); }; + internal::meta_dtor_node node{+[](void *instance) { std::invoke(Func, *static_cast(instance)); }}; + internal::meta_extend(owner(), std::move(node)); return *this; } @@ -298,8 +365,8 @@ public: &meta_setter, &meta_getter}; - auto it = owner().details->data.insert_or_assign(id, std::move(node)).first; - return meta_factory{it->second}; + auto &&elem = internal::meta_extend(owner(), id, std::move(node)); + return meta_factory{elem}; } else { using data_type = std::remove_reference_t>; @@ -311,8 +378,8 @@ public: &meta_setter, &meta_getter}; - auto it = owner().details->data.insert_or_assign(id, std::move(node)).first; - return meta_factory{it->second}; + auto &&elem = internal::meta_extend(owner(), id, std::move(node)); + return meta_factory{elem}; } } @@ -351,8 +418,8 @@ public: &meta_setter, &meta_getter}; - auto it = owner().details->data.insert_or_assign(id, std::move(node)).first; - return meta_factory{it->second}; + auto &&elem = internal::meta_extend(owner(), id, std::move(node)); + return meta_factory{elem}; } else { using args_type = typename meta_function_helper_t::args_type; @@ -365,8 +432,8 @@ public: &meta_setter, &meta_getter}; - auto it = owner().details->data.insert_or_assign(id, std::move(node)).first; - return meta_factory{it->second}; + auto &&elem = internal::meta_extend(owner(), id, std::move(node)); + return meta_factory{elem}; } } @@ -409,7 +476,6 @@ public: auto func(const id_type id) noexcept { using descriptor = meta_function_helper_t; static_assert(Policy::template value, "Invalid return type for the given policy"); - auto &&elem = owner(); internal::meta_func_node node{ (descriptor::is_const ? internal::meta_traits::is_const : internal::meta_traits::is_none) | (descriptor::is_static ? internal::meta_traits::is_static : internal::meta_traits::is_none), @@ -418,22 +484,8 @@ public: &meta_arg, &meta_invoke}; - if(auto it = elem.details->func.find(id); it != elem.details->func.end()) { - for(auto *curr = &it->second; curr; curr = curr->next.get()) { - if(curr->invoke == node.invoke) { - node.next = std::move(curr->next); - *curr = std::move(node); - - return meta_factory{*curr}; - } - } - - // locally overloaded function - node.next = std::make_shared(std::move(elem.details->func[id])); - } - - elem.details->func.insert_or_assign(id, std::move(node)); - return meta_factory{elem.details->func[id]}; + auto &&elem = internal::meta_extend(owner(), id, std::move(node)); + return meta_factory{elem}; } };