diff --git a/src/entt/entity/registry.hpp b/src/entt/entity/registry.hpp index 2f3dcaf37..460931872 100644 --- a/src/entt/entity/registry.hpp +++ b/src/entt/entity/registry.hpp @@ -646,13 +646,6 @@ public: return assure().emplace(*this, entity, std::forward(args)...); } - /*! @copydoc emplace */ - template - [[deprecated("use ::emplace instead")]] - decltype(auto) assign(const entity_type entity, Args &&... args) { - return emplace(entity, std::forward(args)...); - } - /** * @brief Assigns each entity in a range the given component. * @@ -670,14 +663,6 @@ public: assure().insert(*this, first, last, value); } - /*! @copydoc insert */ - template - [[deprecated("use ::insert instead")]] - std::enable_if_t::value_type, entity_type>, void> - assign(It first, It last, const Component &value = {}) { - return insert(std::move(first), std::move(last), value); - } - /** * @brief Assigns each entity in a range the given components. * @@ -698,14 +683,6 @@ public: assure().insert(*this, first, last, from, to); } - /*! @copydoc insert */ - template - [[deprecated("use ::insert instead")]] - std::enable_if_t::value_type, entity_type>, void> - assign(EIt first, EIt last, CIt value) { - return insert(std::move(first), std::move(last), value, value + std::distance(first, last)); - } - /** * @brief Assigns or replaces the given component for an entity. * @@ -738,13 +715,6 @@ public: : cpool.emplace(*this, entity, std::forward(args)...); } - /*! @copydoc emplace_or_replace */ - template - [[deprecated("use ::emplace_or_replace instead")]] - decltype(auto) assign_or_replace(const entity_type entity, Args &&... args) { - return emplace_or_replace(entity, std::forward(args)...); - } - /** * @brief Patches the given component for an entity. * @@ -778,14 +748,6 @@ public: return assure().patch(*this, entity, std::forward(func)...); } - /*! @copydoc patch */ - template - [[deprecated("use registry::patch instead")]] - auto replace(const entity_type entity, Func &&... func) - -> decltype((func(std::declval()), ...), assign(entity)) { - return patch(entity, std::forward(func)...); - } - /** * @brief Replaces the given component for an entity. * @@ -807,8 +769,7 @@ public: * @return A reference to the component being replaced. */ template - auto replace(const entity_type entity, Args &&... args) - -> decltype(std::enable_if_t(), Component{std::forward(args)...}, assure().get(entity)) { + decltype(auto) replace(const entity_type entity, Args &&... args) { return assure().replace(*this, entity, Component{std::forward(args)...}); } @@ -1009,13 +970,6 @@ public: return cpool.contains(entity) ? cpool.get(entity) : cpool.emplace(*this, entity, std::forward(args)...); } - /*! @copydoc get_or_emplace */ - template - [[deprecated("use ::get_or_emplace instead")]] - decltype(auto) get_or_assign(const entity_type entity, Args &&... args) { - return get_or_emplace(entity, std::forward(args)...); - } - /** * @brief Returns pointers to the given components for an entity. * @@ -1190,13 +1144,6 @@ public: return assure().on_update(); } - /*! @copydoc on_update */ - template - [[deprecated("use registry::on_update instead")]] - auto on_replace() { - return on_update(); - } - /** * @brief Returns a sink object for the given component. * diff --git a/test/benchmark/benchmark.cpp b/test/benchmark/benchmark.cpp index b111fcf3e..45784e2f8 100644 --- a/test/benchmark/benchmark.cpp +++ b/test/benchmark/benchmark.cpp @@ -38,9 +38,9 @@ void pathological(Func func) { for(std::uint64_t i = 0; i < 500000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } for(auto i = 0; i < 10; ++i) { @@ -53,9 +53,9 @@ void pathological(Func func) { for(std::uint64_t j = 0; j < 50000L; j++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } } @@ -89,19 +89,19 @@ TEST(Benchmark, CreateMany) { timer.elapsed(); } -TEST(Benchmark, CreateManyAndAssignComponents) { +TEST(Benchmark, CreateManyAndEmplaceComponents) { entt::registry registry; std::vector entities(1000000); - std::cout << "Creating 1000000 entities at once and assign components" << std::endl; + std::cout << "Creating 1000000 entities at once and emplace components" << std::endl; timer timer; registry.create(entities.begin(), entities.end()); for(const auto entity: entities) { - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); } timer.elapsed(); @@ -115,8 +115,8 @@ TEST(Benchmark, CreateManyWithComponents) { timer timer; registry.create(entities.begin(), entities.end()); - registry.assign(entities.begin(), entities.end()); - registry.assign(entities.begin(), entities.end()); + registry.insert(entities.begin(), entities.end()); + registry.insert(entities.begin(), entities.end()); timer.elapsed(); } @@ -145,7 +145,7 @@ TEST(Benchmark, IterateSingleComponent1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -166,7 +166,7 @@ TEST(Benchmark, IterateSingleComponentRuntime1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -189,8 +189,8 @@ TEST(Benchmark, IterateTwoComponents1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -211,10 +211,10 @@ TEST(Benchmark, IterateTwoComponents1MHalf) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -236,10 +236,10 @@ TEST(Benchmark, IterateTwoComponents1MOne) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); if(i == 500000L) { - registry.assign(entity); + registry.emplace(entity); } } @@ -262,8 +262,8 @@ TEST(Benchmark, IterateTwoComponentsNonOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -285,8 +285,8 @@ TEST(Benchmark, IterateTwoComponentsFullOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -308,8 +308,8 @@ TEST(Benchmark, IterateTwoComponentsPartialOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -330,8 +330,8 @@ TEST(Benchmark, IterateTwoComponentsRuntime1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); } auto test = [®istry](auto func) { @@ -358,10 +358,10 @@ TEST(Benchmark, IterateTwoComponentsRuntime1MHalf) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -389,10 +389,10 @@ TEST(Benchmark, IterateTwoComponentsRuntime1MOne) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); if(i == 500000L) { - registry.assign(entity); + registry.emplace(entity); } } @@ -420,9 +420,9 @@ TEST(Benchmark, IterateThreeComponents1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -443,11 +443,11 @@ TEST(Benchmark, IterateThreeComponents1MHalf) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -469,11 +469,11 @@ TEST(Benchmark, IterateThreeComponents1MOne) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); if(i == 500000L) { - registry.assign(entity); + registry.emplace(entity); } } @@ -496,9 +496,9 @@ TEST(Benchmark, IterateThreeComponentsNonOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -520,9 +520,9 @@ TEST(Benchmark, IterateThreeComponentsFullOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -544,9 +544,9 @@ TEST(Benchmark, IterateThreeComponentsPartialOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -567,9 +567,9 @@ TEST(Benchmark, IterateThreeComponentsRuntime1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -598,11 +598,11 @@ TEST(Benchmark, IterateThreeComponentsRuntime1MHalf) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -632,11 +632,11 @@ TEST(Benchmark, IterateThreeComponentsRuntime1MOne) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); if(i == 500000L) { - registry.assign(entity); + registry.emplace(entity); } } @@ -666,11 +666,11 @@ TEST(Benchmark, IterateFiveComponents1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -691,13 +691,13 @@ TEST(Benchmark, IterateFiveComponents1MHalf) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -719,13 +719,13 @@ TEST(Benchmark, IterateFiveComponents1MOne) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); if(i == 500000L) { - registry.assign(entity); + registry.emplace(entity); } } @@ -748,11 +748,11 @@ TEST(Benchmark, IterateFiveComponentsNonOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -774,11 +774,11 @@ TEST(Benchmark, IterateFiveComponentsFullOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -800,11 +800,11 @@ TEST(Benchmark, IterateFiveComponentsPartialFourOfFiveOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -826,11 +826,11 @@ TEST(Benchmark, IterateFiveComponentsPartialThreeOfFiveOwningGroup1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -851,11 +851,11 @@ TEST(Benchmark, IterateFiveComponentsRuntime1M) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); } auto test = [®istry](auto func) { @@ -888,13 +888,13 @@ TEST(Benchmark, IterateFiveComponentsRuntime1MHalf) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -928,13 +928,13 @@ TEST(Benchmark, IterateFiveComponentsRuntime1MOne) { for(std::uint64_t i = 0; i < 1000000L; i++) { const auto entity = registry.create(); - registry.assign(entity); - registry.assign>(entity); - registry.assign>(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace>(entity); + registry.emplace>(entity); + registry.emplace>(entity); if(i == 500000L) { - registry.assign(entity); + registry.emplace(entity); } } @@ -1014,7 +1014,7 @@ TEST(Benchmark, SortSingle) { for(std::uint64_t i = 0; i < 150000L; i++) { const auto entity = registry.create(); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); } timer timer; @@ -1033,8 +1033,8 @@ TEST(Benchmark, SortMulti) { for(std::uint64_t i = 0; i < 150000L; i++) { const auto entity = registry.create(); - registry.assign(entity, i, i); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); + registry.emplace(entity, i, i); } registry.sort([](const auto &lhs, const auto &rhs) { @@ -1056,7 +1056,7 @@ TEST(Benchmark, AlmostSortedStdSort) { for(std::uint64_t i = 0; i < 150000L; i++) { const auto entity = registry.create(); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); if(!(i % 50000)) { entities[i / 50000] = entity; @@ -1066,7 +1066,7 @@ TEST(Benchmark, AlmostSortedStdSort) { for(std::uint64_t i = 0; i < 3; ++i) { registry.destroy(entities[i]); const auto entity = registry.create(); - registry.assign(entity, 50000 * i, 50000 * i); + registry.emplace(entity, 50000 * i, 50000 * i); } timer timer; @@ -1086,7 +1086,7 @@ TEST(Benchmark, AlmostSortedInsertionSort) { for(std::uint64_t i = 0; i < 150000L; i++) { const auto entity = registry.create(); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); if(!(i % 50000)) { entities[i / 50000] = entity; @@ -1096,7 +1096,7 @@ TEST(Benchmark, AlmostSortedInsertionSort) { for(std::uint64_t i = 0; i < 3; ++i) { registry.destroy(entities[i]); const auto entity = registry.create(); - registry.assign(entity, 50000 * i, 50000 * i); + registry.emplace(entity, 50000 * i, 50000 * i); } timer timer; diff --git a/test/entt/entity/actor.cpp b/test/entt/entity/actor.cpp index ff4e8f79f..8a55619ac 100644 --- a/test/entt/entity/actor.cpp +++ b/test/entt/entity/actor.cpp @@ -43,8 +43,8 @@ TEST(Actor, FromEntity) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity, 42); - registry.assign(entity, 'c'); + registry.emplace(entity, 42); + registry.emplace(entity, 'c'); entt::actor actor{entity, registry}; diff --git a/test/entt/entity/entity.cpp b/test/entt/entity/entity.cpp index d78d8a22d..d65af3af1 100644 --- a/test/entt/entity/entity.cpp +++ b/test/entt/entity/entity.cpp @@ -10,7 +10,7 @@ TEST(Entity, Null) { entt::registry registry{}; const auto entity = registry.create(); - registry.assign(entity, 42); + registry.emplace(entity, 42); ASSERT_FALSE(entt::entity{} == entt::null); ASSERT_TRUE(entt::entity{traits_type::entity_mask} == entt::null); diff --git a/test/entt/entity/group.cpp b/test/entt/entity/group.cpp index d77db0b00..d1ffd0d7f 100644 --- a/test/entt/entity/group.cpp +++ b/test/entt/entity/group.cpp @@ -24,11 +24,11 @@ TEST(NonOwningGroup, Functionalities) { ASSERT_TRUE((cgroup.empty())); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_FALSE(group.empty()); ASSERT_FALSE((group.empty())); @@ -42,7 +42,7 @@ TEST(NonOwningGroup, Functionalities) { ASSERT_EQ(group.size(), typename decltype(group)::size_type{1}); ASSERT_EQ(cgroup.size(), typename decltype(group)::size_type{2}); - registry.assign(e0); + registry.emplace(e0); ASSERT_EQ(group.size(), typename decltype(group)::size_type{2}); ASSERT_EQ(group.size(), typename decltype(group)::size_type{2}); @@ -94,12 +94,12 @@ TEST(NonOwningGroup, ElementAccess) { auto cgroup = std::as_const(registry).group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); for(typename decltype(group)::size_type i{}; i < group.size(); ++i) { ASSERT_EQ(group[i], i ? e0 : e1); @@ -112,12 +112,12 @@ TEST(NonOwningGroup, Contains) { auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); registry.destroy(e0); @@ -129,13 +129,13 @@ TEST(NonOwningGroup, Empty) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_TRUE(registry.group(entt::get).empty()); ASSERT_TRUE(registry.group(entt::get).empty()); @@ -146,12 +146,12 @@ TEST(NonOwningGroup, Each) { auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); auto cgroup = std::as_const(registry).group(entt::get); std::size_t cnt = 0; @@ -176,14 +176,14 @@ TEST(NonOwningGroup, Sort) { const auto e2 = registry.create(); const auto e3 = registry.create(); - registry.assign(e0, 0u); - registry.assign(e1, 1u); - registry.assign(e2, 2u); - registry.assign(e3, 3u); + registry.emplace(e0, 0u); + registry.emplace(e1, 1u); + registry.emplace(e2, 2u); + registry.emplace(e3, 3u); - registry.assign(e0, 0); - registry.assign(e1, 1); - registry.assign(e2, 2); + registry.emplace(e0, 0); + registry.emplace(e1, 1); + registry.emplace(e2, 2); ASSERT_EQ(*(group.raw() + 0u), 0u); ASSERT_EQ(*(group.raw() + 1u), 1u); @@ -246,14 +246,14 @@ TEST(NonOwningGroup, SortAsAPool) { auto uval = 0u; auto ival = 0; - registry.assign(e0, uval++); - registry.assign(e1, uval++); - registry.assign(e2, uval++); - registry.assign(e3, uval+1); + registry.emplace(e0, uval++); + registry.emplace(e1, uval++); + registry.emplace(e2, uval++); + registry.emplace(e3, uval+1); - registry.assign(e0, ival++); - registry.assign(e1, ival++); - registry.assign(e2, ival++); + registry.emplace(e0, ival++); + registry.emplace(e1, ival++); + registry.emplace(e2, ival++); for(auto entity: group) { ASSERT_EQ(group.get(entity), --uval); @@ -282,14 +282,14 @@ TEST(NonOwningGroup, IndexRebuiltOnDestroy) { const auto e0 = registry.create(); const auto e1 = registry.create(); - registry.assign(e0, 0u); - registry.assign(e1, 1u); + registry.emplace(e0, 0u); + registry.emplace(e1, 1u); - registry.assign(e0, 0); - registry.assign(e1, 1); + registry.emplace(e0, 0); + registry.emplace(e1, 1); registry.destroy(e0); - registry.assign(registry.create(), 42); + registry.emplace(registry.create(), 42); ASSERT_EQ(group.size(), typename decltype(group)::size_type{1}); ASSERT_EQ(group[{}], e1); @@ -310,8 +310,8 @@ TEST(NonOwningGroup, ConstNonConstAndAllInBetween) { ASSERT_EQ(group.size(), decltype(group.size()){0}); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), decltype(group.size()){1}); @@ -332,20 +332,20 @@ TEST(NonOwningGroup, Find) { auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); const auto e2 = registry.create(); - registry.assign(e2); - registry.assign(e2); + registry.emplace(e2); + registry.emplace(e2); const auto e3 = registry.create(); - registry.assign(e3); - registry.assign(e3); + registry.emplace(e3); + registry.emplace(e3); registry.remove(e1); @@ -365,8 +365,8 @@ TEST(NonOwningGroup, Find) { const auto e4 = registry.create(); registry.destroy(e4); const auto e5 = registry.create(); - registry.assign(e5); - registry.assign(e5); + registry.emplace(e5); + registry.emplace(e5); ASSERT_NE(group.find(e5), group.end()); ASSERT_EQ(group.find(e4), group.end()); @@ -376,20 +376,20 @@ TEST(NonOwningGroup, ExcludedComponents) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0, 0); + registry.emplace(e0, 0); const auto e1 = registry.create(); - registry.assign(e1, 1); - registry.assign(e1); + registry.emplace(e1, 1); + registry.emplace(e1); const auto group = registry.group(entt::get, entt::exclude); const auto e2 = registry.create(); - registry.assign(e2, 2); + registry.emplace(e2, 2); const auto e3 = registry.create(); - registry.assign(e3, 3); - registry.assign(e3); + registry.emplace(e3, 3); + registry.emplace(e3); for(const auto entity: group) { ASSERT_TRUE(entity == e0 || entity == e2); @@ -401,8 +401,8 @@ TEST(NonOwningGroup, ExcludedComponents) { } } - registry.assign(e0); - registry.assign(e2); + registry.emplace(e0); + registry.emplace(e2); ASSERT_TRUE(group.empty()); @@ -425,14 +425,14 @@ TEST(NonOwningGroup, EmptyAndNonEmptyTypes) { const auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); - registry.assign(registry.create()); + registry.emplace(registry.create()); for(const auto entity: group) { ASSERT_TRUE(entity == e0 || entity == e1); @@ -451,8 +451,8 @@ TEST(NonOwningGroup, TrackEntitiesOnComponentDestruction) { const auto cgroup = std::as_const(registry).group(entt::get, entt::exclude); const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); ASSERT_TRUE(group.empty()); ASSERT_TRUE(cgroup.empty()); @@ -467,9 +467,9 @@ TEST(NonOwningGroup, Less) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); registry.group(entt::get>).less([entity](const auto entt, int, char) { ASSERT_EQ(entity, entt); @@ -495,15 +495,15 @@ TEST(NonOwningGroup, FrontBack) { ASSERT_EQ(group.back(), static_cast(entt::null)); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(group.front(), e1); ASSERT_EQ(group.back(), e0); @@ -511,11 +511,11 @@ TEST(NonOwningGroup, FrontBack) { TEST(NonOwningGroup, SignalRace) { entt::registry registry; - registry.on_construct().connect<&entt::registry::assign_or_replace>(); + registry.on_construct().connect<&entt::registry::emplace_or_replace>(); registry.group(entt::get); auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(registry.group(entt::get).size(), 1u); } @@ -530,11 +530,11 @@ TEST(OwningGroup, Functionalities) { ASSERT_TRUE((cgroup.empty())); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_FALSE(group.empty()); ASSERT_FALSE((group.empty())); @@ -548,7 +548,7 @@ TEST(OwningGroup, Functionalities) { ASSERT_EQ(group.size(), typename decltype(group)::size_type{1}); ASSERT_EQ(cgroup.size(), typename decltype(group)::size_type{2}); - registry.assign(e0); + registry.emplace(e0); ASSERT_EQ(group.size(), typename decltype(group)::size_type{2}); ASSERT_EQ(group.size(), typename decltype(group)::size_type{2}); @@ -597,12 +597,12 @@ TEST(OwningGroup, ElementAccess) { auto cgroup = std::as_const(registry).group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); for(typename decltype(group)::size_type i{}; i < group.size(); ++i) { ASSERT_EQ(group[i], i ? e0 : e1); @@ -615,12 +615,12 @@ TEST(OwningGroup, Contains) { auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); registry.destroy(e0); @@ -632,13 +632,13 @@ TEST(OwningGroup, Empty) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_TRUE((registry.group(entt::get).empty())); ASSERT_TRUE((registry.group(entt::get).empty())); @@ -649,12 +649,12 @@ TEST(OwningGroup, Each) { auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); auto cgroup = std::as_const(registry).group(entt::get); std::size_t cnt = 0; @@ -677,17 +677,17 @@ TEST(OwningGroup, SortOrdered) { entt::entity entities[5]{}; registry.create(std::begin(entities), std::end(entities)); - registry.assign(entities[0], 12); - registry.assign(entities[0], 'a'); + registry.emplace(entities[0], 12); + registry.emplace(entities[0], 'a'); - registry.assign(entities[1], 9); - registry.assign(entities[1], 'b'); + registry.emplace(entities[1], 9); + registry.emplace(entities[1], 'b'); - registry.assign(entities[2], 6); - registry.assign(entities[2], 'c'); + registry.emplace(entities[2], 6); + registry.emplace(entities[2], 'c'); - registry.assign(entities[3], 1); - registry.assign(entities[4], 2); + registry.emplace(entities[3], 1); + registry.emplace(entities[4], 2); group.sort([&group](const entt::entity lhs, const entt::entity rhs) { return group.get(lhs).value < group.get(rhs).value; @@ -724,17 +724,17 @@ TEST(OwningGroup, SortReverse) { entt::entity entities[5]{}; registry.create(std::begin(entities), std::end(entities)); - registry.assign(entities[0], 6); - registry.assign(entities[0], 'a'); + registry.emplace(entities[0], 6); + registry.emplace(entities[0], 'a'); - registry.assign(entities[1], 9); - registry.assign(entities[1], 'b'); + registry.emplace(entities[1], 9); + registry.emplace(entities[1], 'b'); - registry.assign(entities[2], 12); - registry.assign(entities[2], 'c'); + registry.emplace(entities[2], 12); + registry.emplace(entities[2], 'c'); - registry.assign(entities[3], 1); - registry.assign(entities[4], 2); + registry.emplace(entities[3], 1); + registry.emplace(entities[4], 2); group.sort([](const auto &lhs, const auto &rhs) { return lhs.value < rhs.value; @@ -771,23 +771,23 @@ TEST(OwningGroup, SortUnordered) { entt::entity entities[7]{}; registry.create(std::begin(entities), std::end(entities)); - registry.assign(entities[0], 6); - registry.assign(entities[0], 'c'); + registry.emplace(entities[0], 6); + registry.emplace(entities[0], 'c'); - registry.assign(entities[1], 3); - registry.assign(entities[1], 'b'); + registry.emplace(entities[1], 3); + registry.emplace(entities[1], 'b'); - registry.assign(entities[2], 1); - registry.assign(entities[2], 'a'); + registry.emplace(entities[2], 1); + registry.emplace(entities[2], 'a'); - registry.assign(entities[3], 9); - registry.assign(entities[3], 'd'); + registry.emplace(entities[3], 9); + registry.emplace(entities[3], 'd'); - registry.assign(entities[4], 12); - registry.assign(entities[4], 'e'); + registry.emplace(entities[4], 12); + registry.emplace(entities[4], 'e'); - registry.assign(entities[5], 4); - registry.assign(entities[6], 5); + registry.emplace(entities[5], 4); + registry.emplace(entities[6], 5); group.sort([](const auto lhs, const auto rhs) { return lhs < rhs; @@ -832,13 +832,13 @@ TEST(OwningGroup, SortWithExclusionList) { entt::entity entities[5]{}; registry.create(std::begin(entities), std::end(entities)); - registry.assign(entities[0], 0); - registry.assign(entities[1], 1); - registry.assign(entities[2], 2); - registry.assign(entities[3], 3); - registry.assign(entities[4], 4); + registry.emplace(entities[0], 0); + registry.emplace(entities[1], 1); + registry.emplace(entities[2], 2); + registry.emplace(entities[3], 3); + registry.emplace(entities[4], 4); - registry.assign(entities[2]); + registry.emplace(entities[2]); group.sort([](const entt::entity lhs, const entt::entity rhs) { return lhs < rhs; @@ -869,14 +869,14 @@ TEST(OwningGroup, IndexRebuiltOnDestroy) { const auto e0 = registry.create(); const auto e1 = registry.create(); - registry.assign(e0, 0u); - registry.assign(e1, 1u); + registry.emplace(e0, 0u); + registry.emplace(e1, 1u); - registry.assign(e0, 0); - registry.assign(e1, 1); + registry.emplace(e0, 0); + registry.emplace(e1, 1); registry.destroy(e0); - registry.assign(registry.create(), 42); + registry.emplace(registry.create(), 42); ASSERT_EQ(group.size(), typename decltype(group)::size_type{1}); ASSERT_EQ(group[{}], e1); @@ -897,10 +897,10 @@ TEST(OwningGroup, ConstNonConstAndAllInBetween) { ASSERT_EQ(group.size(), decltype(group.size()){0}); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); - registry.assign(entity, 0.); - registry.assign(entity, 0.f); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); + registry.emplace(entity, 0.); + registry.emplace(entity, 0.f); ASSERT_EQ(group.size(), decltype(group.size()){1}); @@ -927,20 +927,20 @@ TEST(OwningGroup, Find) { auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); const auto e2 = registry.create(); - registry.assign(e2); - registry.assign(e2); + registry.emplace(e2); + registry.emplace(e2); const auto e3 = registry.create(); - registry.assign(e3); - registry.assign(e3); + registry.emplace(e3); + registry.emplace(e3); registry.remove(e1); @@ -960,8 +960,8 @@ TEST(OwningGroup, Find) { const auto e4 = registry.create(); registry.destroy(e4); const auto e5 = registry.create(); - registry.assign(e5); - registry.assign(e5); + registry.emplace(e5); + registry.emplace(e5); ASSERT_NE(group.find(e5), group.end()); ASSERT_EQ(group.find(e4), group.end()); @@ -971,20 +971,20 @@ TEST(OwningGroup, ExcludedComponents) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0, 0); + registry.emplace(e0, 0); const auto e1 = registry.create(); - registry.assign(e1, 1); - registry.assign(e1); + registry.emplace(e1, 1); + registry.emplace(e1); const auto group = registry.group(entt::exclude); const auto e2 = registry.create(); - registry.assign(e2, 2); + registry.emplace(e2, 2); const auto e3 = registry.create(); - registry.assign(e3, 3); - registry.assign(e3); + registry.emplace(e3, 3); + registry.emplace(e3); for(const auto entity: group) { ASSERT_TRUE(entity == e0 || entity == e2); @@ -996,8 +996,8 @@ TEST(OwningGroup, ExcludedComponents) { } } - registry.assign(e0); - registry.assign(e2); + registry.emplace(e0); + registry.emplace(e2); ASSERT_TRUE(group.empty()); @@ -1020,14 +1020,14 @@ TEST(OwningGroup, EmptyAndNonEmptyTypes) { const auto group = registry.group(entt::get); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); - registry.assign(registry.create()); + registry.emplace(registry.create()); for(const auto entity: group) { ASSERT_TRUE(entity == e0 || entity == e1); @@ -1046,8 +1046,8 @@ TEST(OwningGroup, TrackEntitiesOnComponentDestruction) { const auto cgroup = std::as_const(registry).group(entt::exclude); const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); ASSERT_TRUE(group.empty()); ASSERT_TRUE(cgroup.empty()); @@ -1062,9 +1062,9 @@ TEST(OwningGroup, Less) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); registry.group(entt::get>).less([entity](const auto entt, int, char) { ASSERT_EQ(entity, entt); @@ -1090,15 +1090,15 @@ TEST(OwningGroup, FrontBack) { ASSERT_EQ(group.back(), static_cast(entt::null)); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(group.front(), e1); ASSERT_EQ(group.back(), e0); @@ -1106,11 +1106,11 @@ TEST(OwningGroup, FrontBack) { TEST(OwningGroup, SignalRace) { entt::registry registry; - registry.on_construct().connect<&entt::registry::assign_or_replace>(); + registry.on_construct().connect<&entt::registry::emplace_or_replace>(); registry.group(entt::get); auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(registry.group(entt::get).size(), 1u); } @@ -1120,8 +1120,8 @@ TEST(OwningGroup, StableLateInitialization) { for(std::size_t i{}; i < 30u; ++i) { auto entity = registry.create(); - if(!(i % 2u)) registry.assign(entity); - if(!(i % 3u)) registry.assign(entity); + if(!(i % 2u)) registry.emplace(entity); + if(!(i % 3u)) registry.emplace(entity); } // thanks to @pgruenbacher for pointing out this corner case @@ -1131,11 +1131,11 @@ TEST(OwningGroup, StableLateInitialization) { TEST(OwningGroup, PreventEarlyOptOut) { entt::registry registry; - registry.assign(registry.create(), 3); + registry.emplace(registry.create(), 3); const auto entity = registry.create(); - registry.assign(entity, 'c'); - registry.assign(entity, 2); + registry.emplace(entity, 'c'); + registry.emplace(entity, 2); // thanks to @pgruenbacher for pointing out this corner case registry.group().each([entity](const auto entt, const auto &c, const auto &i) { diff --git a/test/entt/entity/helper.cpp b/test/entt/entity/helper.cpp index c4957bd0a..2a7c2c5a2 100644 --- a/test/entt/entity/helper.cpp +++ b/test/entt/entity/helper.cpp @@ -34,7 +34,7 @@ TEST(Invoke, MemberFunction) { const auto entity = registry.create(); registry.on_construct().connect>(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(entity, registry.get(entity).entt); } diff --git a/test/entt/entity/observer.cpp b/test/entt/entity/observer.cpp index aa8c77779..9c429c7c7 100644 --- a/test/entt/entity/observer.cpp +++ b/test/entt/entity/observer.cpp @@ -15,7 +15,7 @@ TEST(Observer, Functionalities) { ASSERT_EQ(observer.begin(), observer.end()); const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); @@ -32,7 +32,7 @@ TEST(Observer, Functionalities) { observer.disconnect(); registry.remove(entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(observer.size(), 0u); ASSERT_TRUE(observer.empty()); @@ -49,14 +49,14 @@ TEST(Observer, AllOf) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); ASSERT_EQ(*observer.data(), entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_FALSE(observer.empty()); @@ -64,8 +64,8 @@ TEST(Observer, AllOf) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); ASSERT_FALSE(observer.empty()); @@ -73,14 +73,14 @@ TEST(Observer, AllOf) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); + registry.emplace(entity); observer.clear(); ASSERT_TRUE(observer.empty()); observer.disconnect(); - registry.assign_or_replace(entity); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); + registry.emplace_or_replace(entity); registry.remove_if_exists(entity); ASSERT_TRUE(observer.empty()); @@ -96,28 +96,28 @@ TEST(Observer, AllOfFiltered) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(observer.size(), 0u); ASSERT_TRUE(observer.empty()); ASSERT_EQ(observer.data(), nullptr); registry.remove(entity); - registry.assign(entity); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); registry.remove(entity); registry.remove(entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); ASSERT_EQ(*observer.data(), entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); @@ -127,7 +127,7 @@ TEST(Observer, AllOfFiltered) { observer.disconnect(); registry.remove(entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); } @@ -139,12 +139,12 @@ TEST(Observer, Observe) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); @@ -160,8 +160,8 @@ TEST(Observer, Observe) { ASSERT_TRUE(observer.empty()); observer.disconnect(); - registry.assign_or_replace(entity); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_TRUE(observer.empty()); } @@ -176,15 +176,15 @@ TEST(Observer, ObserveFiltered) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); + registry.emplace(entity); registry.replace(entity); ASSERT_EQ(observer.size(), 0u); ASSERT_TRUE(observer.empty()); ASSERT_EQ(observer.data(), nullptr); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); registry.replace(entity); ASSERT_TRUE(observer.empty()); @@ -196,7 +196,7 @@ TEST(Observer, ObserveFiltered) { ASSERT_FALSE(observer.empty()); ASSERT_EQ(*observer.data(), entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); @@ -219,8 +219,8 @@ TEST(Observer, AllOfObserve) { ASSERT_TRUE(observer.empty()); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); registry.replace(entity); registry.remove(entity); @@ -229,7 +229,7 @@ TEST(Observer, AllOfObserve) { ASSERT_EQ(*observer.data(), entity); registry.remove(entity); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); @@ -239,8 +239,8 @@ TEST(Observer, AllOfObserve) { ASSERT_TRUE(observer.empty()); observer.disconnect(); - registry.assign_or_replace(entity); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_TRUE(observer.empty()); } @@ -250,12 +250,12 @@ TEST(Observer, CrossRulesCornerCase) { entt::observer observer{registry, entt::collector.group().group()}; const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); observer.clear(); ASSERT_TRUE(observer.empty()); - registry.assign(entity); + registry.emplace(entity); registry.remove(entity); ASSERT_FALSE(observer.empty()); @@ -265,7 +265,7 @@ TEST(Observer, Each) { entt::registry registry; entt::observer observer{registry, entt::collector.group()}; const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_FALSE(observer.empty()); ASSERT_EQ(observer.size(), 1u); @@ -296,28 +296,28 @@ TEST(Observer, MultipleFilters) { ASSERT_TRUE(observer.empty()); - registry.assign_or_replace(entity); - registry.assign(entity); + registry.emplace_or_replace(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); ASSERT_EQ(*observer.data(), entity); observer.clear(); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); - registry.assign_or_replace(entity); - registry.assign(entity); + registry.emplace_or_replace(entity); + registry.emplace(entity); ASSERT_TRUE(observer.empty()); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); @@ -327,7 +327,7 @@ TEST(Observer, MultipleFilters) { ASSERT_TRUE(observer.empty()); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_EQ(observer.size(), 1u); ASSERT_FALSE(observer.empty()); @@ -336,7 +336,7 @@ TEST(Observer, MultipleFilters) { observer.clear(); observer.disconnect(); - registry.assign_or_replace(entity); + registry.emplace_or_replace(entity); ASSERT_TRUE(observer.empty()); } diff --git a/test/entt/entity/registry.cpp b/test/entt/entity/registry.cpp index e221fb660..bfea2a3b8 100644 --- a/test/entt/entity/registry.cpp +++ b/test/entt/entity/registry.cpp @@ -121,8 +121,8 @@ TEST(Registry, Functionalities) { const auto e0 = registry.create(); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_TRUE(registry.has<>(e0)); ASSERT_FALSE(registry.any<>(e1)); @@ -146,8 +146,8 @@ TEST(Registry, Functionalities) { ASSERT_EQ(registry.try_get(e0), nullptr); ASSERT_EQ(registry.try_get(e1), nullptr); - ASSERT_EQ(registry.assign(e0, 42), 42); - ASSERT_EQ(registry.assign(e0, 'c'), 'c'); + ASSERT_EQ(registry.emplace(e0, 42), 42); + ASSERT_EQ(registry.emplace(e0, 'c'), 'c'); ASSERT_NO_THROW(registry.remove(e1)); ASSERT_NO_THROW(registry.remove(e1)); @@ -158,8 +158,8 @@ TEST(Registry, Functionalities) { const auto e2 = registry.create(); - registry.assign_or_replace(e2, registry.get(e0)); - registry.assign_or_replace(e2, registry.get(e0)); + registry.emplace_or_replace(e2, registry.get(e0)); + registry.emplace_or_replace(e2, registry.get(e0)); ASSERT_TRUE((registry.has(e2))); ASSERT_EQ(registry.get(e0), 42); @@ -181,11 +181,11 @@ TEST(Registry, Functionalities) { ASSERT_NE(®istry.get(e0), ®istry.get(e2)); ASSERT_NE(®istry.get(e0), ®istry.get(e2)); - ASSERT_EQ(registry.replace(e0, [](auto &instance) { instance = 2; }), 2); + ASSERT_EQ(registry.patch(e0, [](auto &instance) { instance = 2; }), 2); ASSERT_EQ(registry.replace(e0, 3), 3); - ASSERT_NO_THROW(registry.assign_or_replace(e0, 1)); - ASSERT_NO_THROW(registry.assign_or_replace(e1, 1)); + ASSERT_NO_THROW(registry.emplace_or_replace(e0, 1)); + ASSERT_NO_THROW(registry.emplace_or_replace(e1, 1)); ASSERT_EQ(static_cast(registry).get(e0), 1); ASSERT_EQ(static_cast(registry).get(e1), 1); @@ -215,8 +215,8 @@ TEST(Registry, Functionalities) { const auto e3 = registry.create(); - ASSERT_EQ(registry.get_or_assign(e3, 3), 3); - ASSERT_EQ(registry.get_or_assign(e3, 'c'), 'c'); + ASSERT_EQ(registry.get_or_emplace(e3, 3), 3); + ASSERT_EQ(registry.get_or_emplace(e3, 'c'), 'c'); ASSERT_EQ(registry.size(), entt::registry::size_type{1}); ASSERT_EQ(registry.size(), entt::registry::size_type{1}); @@ -242,7 +242,7 @@ TEST(Registry, Functionalities) { const auto e4 = registry.create(); const auto e5 = registry.create(); - registry.assign(e4); + registry.emplace(e4); ASSERT_NO_THROW(registry.remove_if_exists(e4)); ASSERT_NO_THROW(registry.remove_if_exists(e5)); @@ -264,16 +264,16 @@ TEST(Registry, ReplaceAggregate) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity, 0); + registry.emplace(entity, 0); auto &instance = registry.replace(entity, 42); ASSERT_EQ(instance.value, 42); } -TEST(Registry, AssignOrReplaceAggregate) { +TEST(Registry, EmplaceOrReplaceAggregate) { entt::registry registry; const auto entity = registry.create(); - auto &instance = registry.assign_or_replace(entity, 42); + auto &instance = registry.emplace_or_replace(entity, 42); ASSERT_EQ(instance.value, 42); } @@ -306,7 +306,7 @@ TEST(Registry, RawData) { ASSERT_EQ(std::as_const(registry).data(), nullptr); ASSERT_EQ(*std::as_const(registry).data(), entity); - registry.assign(entity, 42); + registry.emplace(entity, 42); ASSERT_EQ(*registry.raw(), 42); ASSERT_EQ(*std::as_const(registry).raw(), 42); @@ -351,8 +351,8 @@ TEST(Registry, CreateManyEntitiesAtOnceWithListener) { registry.on_construct().connect<&listener::incr>(listener); registry.create(std::begin(entities), std::end(entities)); - registry.assign(std::begin(entities), std::end(entities), 42); - registry.assign(std::begin(entities), std::end(entities), 'c'); + registry.insert(std::begin(entities), std::end(entities), 42); + registry.insert(std::begin(entities), std::end(entities), 'c'); ASSERT_EQ(registry.get(entities[0]), 42); ASSERT_EQ(registry.get(entities[1]), 'c'); @@ -361,8 +361,8 @@ TEST(Registry, CreateManyEntitiesAtOnceWithListener) { registry.on_construct().disconnect<&listener::incr>(listener); registry.on_construct().connect<&listener::incr>(listener); registry.create(std::begin(entities), std::end(entities)); - registry.assign(std::begin(entities), std::end(entities), 'a'); - registry.assign(std::begin(entities), std::end(entities)); + registry.insert(std::begin(entities), std::end(entities), 'a'); + registry.insert(std::begin(entities), std::end(entities)); ASSERT_TRUE(registry.has(entities[0])); ASSERT_EQ(registry.get(entities[2]), 'a'); @@ -422,14 +422,14 @@ TEST(Registry, CreateDestroyEntities) { for(int i = 0; i < 10; ++i) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); } registry.clear(); for(int i = 0; i < 7; ++i) { const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); if(i == 3) { pre = entity; } } @@ -478,9 +478,9 @@ TEST(Registry, Each) { entt::registry::size_type match; registry.create(); - registry.assign(registry.create()); + registry.emplace(registry.create()); registry.create(); - registry.assign(registry.create()); + registry.emplace(registry.create()); registry.create(); tot = 0u; @@ -529,9 +529,9 @@ TEST(Registry, Orphans) { entt::registry registry; entt::registry::size_type tot{}; - registry.assign(registry.create()); + registry.emplace(registry.create()); registry.create(); - registry.assign(registry.create()); + registry.emplace(registry.create()); registry.orphans([&](auto) { ++tot; }); ASSERT_EQ(tot, 1u); @@ -554,15 +554,15 @@ TEST(Registry, View) { auto cview = registry.view(); const auto e0 = registry.create(); - registry.assign(e0, 0); - registry.assign(e0, 'c'); + registry.emplace(e0, 0); + registry.emplace(e0, 'c'); const auto e1 = registry.create(); - registry.assign(e1, 0); + registry.emplace(e1, 0); const auto e2 = registry.create(); - registry.assign(e2, 0); - registry.assign(e2, 'c'); + registry.emplace(e2, 0); + registry.emplace(e2, 'c'); ASSERT_EQ(iview.size(), 3u); ASSERT_EQ(cview.size(), 2u); @@ -577,7 +577,7 @@ TEST(Registry, NonOwningGroupInitOnFirstUse) { entt::registry registry; auto create = [&](auto... component) { const auto entity = registry.create(); - (registry.assign(entity, component), ...); + (registry.emplace(entity, component), ...); }; create(0, 'c'); @@ -592,12 +592,12 @@ TEST(Registry, NonOwningGroupInitOnFirstUse) { ASSERT_EQ(cnt, 2u); } -TEST(Registry, NonOwningGroupInitOnAssign) { +TEST(Registry, NonOwningGroupInitOnEmplace) { entt::registry registry; auto group = registry.group<>(entt::get); auto create = [&](auto... component) { const auto entity = registry.create(); - (registry.assign(entity, component), ...); + (registry.emplace(entity, component), ...); }; create(0, 'c'); @@ -615,7 +615,7 @@ TEST(Registry, FullOwningGroupInitOnFirstUse) { entt::registry registry; auto create = [&](auto... component) { const auto entity = registry.create(); - (registry.assign(entity, component), ...); + (registry.emplace(entity, component), ...); }; create(0, 'c'); @@ -632,12 +632,12 @@ TEST(Registry, FullOwningGroupInitOnFirstUse) { ASSERT_EQ(cnt, 2u); } -TEST(Registry, FullOwningGroupInitOnAssign) { +TEST(Registry, FullOwningGroupInitOnEmplace) { entt::registry registry; auto group = registry.group(); auto create = [&](auto... component) { const auto entity = registry.create(); - (registry.assign(entity, component), ...); + (registry.emplace(entity, component), ...); }; create(0, 'c'); @@ -657,7 +657,7 @@ TEST(Registry, PartialOwningGroupInitOnFirstUse) { entt::registry registry; auto create = [&](auto... component) { const auto entity = registry.create(); - (registry.assign(entity, component), ...); + (registry.emplace(entity, component), ...); }; create(0, 'c'); @@ -675,12 +675,12 @@ TEST(Registry, PartialOwningGroupInitOnFirstUse) { } -TEST(Registry, PartialOwningGroupInitOnAssign) { +TEST(Registry, PartialOwningGroupInitOnEmplace) { entt::registry registry; auto group = registry.group(entt::get); auto create = [&](auto... component) { const auto entity = registry.create(); - (registry.assign(entity, component), ...); + (registry.emplace(entity, component), ...); }; create(0, 'c'); @@ -701,8 +701,8 @@ TEST(Registry, CleanViewAfterRemoveAndClear) { auto view = registry.view(); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); ASSERT_EQ(view.size(), entt::registry::size_type{1}); @@ -710,7 +710,7 @@ TEST(Registry, CleanViewAfterRemoveAndClear) { ASSERT_EQ(view.size(), entt::registry::size_type{0}); - registry.assign(entity, 'c'); + registry.emplace(entity, 'c'); ASSERT_EQ(view.size(), entt::registry::size_type{1}); @@ -718,7 +718,7 @@ TEST(Registry, CleanViewAfterRemoveAndClear) { ASSERT_EQ(view.size(), entt::registry::size_type{0}); - registry.assign(entity, 0); + registry.emplace(entity, 0); ASSERT_EQ(view.size(), entt::registry::size_type{1}); @@ -732,8 +732,8 @@ TEST(Registry, CleanNonOwningGroupViewAfterRemoveAndClear) { auto group = registry.group<>(entt::get); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -741,7 +741,7 @@ TEST(Registry, CleanNonOwningGroupViewAfterRemoveAndClear) { ASSERT_EQ(group.size(), entt::registry::size_type{0}); - registry.assign(entity, 'c'); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -749,7 +749,7 @@ TEST(Registry, CleanNonOwningGroupViewAfterRemoveAndClear) { ASSERT_EQ(group.size(), entt::registry::size_type{0}); - registry.assign(entity, 0); + registry.emplace(entity, 0); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -763,8 +763,8 @@ TEST(Registry, CleanFullOwningGroupViewAfterRemoveAndClear) { auto group = registry.group(); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -772,7 +772,7 @@ TEST(Registry, CleanFullOwningGroupViewAfterRemoveAndClear) { ASSERT_EQ(group.size(), entt::registry::size_type{0}); - registry.assign(entity, 'c'); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -780,7 +780,7 @@ TEST(Registry, CleanFullOwningGroupViewAfterRemoveAndClear) { ASSERT_EQ(group.size(), entt::registry::size_type{0}); - registry.assign(entity, 0); + registry.emplace(entity, 0); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -794,8 +794,8 @@ TEST(Registry, CleanPartialOwningGroupViewAfterRemoveAndClear) { auto group = registry.group(entt::get); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -803,7 +803,7 @@ TEST(Registry, CleanPartialOwningGroupViewAfterRemoveAndClear) { ASSERT_EQ(group.size(), entt::registry::size_type{0}); - registry.assign(entity, 'c'); + registry.emplace(entity, 'c'); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -811,7 +811,7 @@ TEST(Registry, CleanPartialOwningGroupViewAfterRemoveAndClear) { ASSERT_EQ(group.size(), entt::registry::size_type{0}); - registry.assign(entity, 0); + registry.emplace(entity, 0); ASSERT_EQ(group.size(), entt::registry::size_type{1}); @@ -825,8 +825,8 @@ TEST(Registry, NestedGroups) { entt::entity entities[10]; registry.create(std::begin(entities), std::end(entities)); - registry.assign(std::begin(entities), std::end(entities)); - registry.assign(std::begin(entities), std::end(entities)); + registry.insert(std::begin(entities), std::end(entities)); + registry.insert(std::begin(entities), std::end(entities)); const auto g1 = registry.group(entt::get, entt::exclude); ASSERT_TRUE(g1.sortable()); @@ -844,7 +844,7 @@ TEST(Registry, NestedGroups) { ASSERT_TRUE(g1.contains(entities[i*2])); ASSERT_TRUE(g2.contains(entities[i*2+1])); ASSERT_TRUE(g2.contains(entities[i*2])); - registry.assign(entities[i*2]); + registry.emplace(entities[i*2]); } ASSERT_EQ(g1.size(), 5u); @@ -878,7 +878,7 @@ TEST(Registry, NestedGroups) { ASSERT_TRUE(g2.contains(entities[i*2])); ASSERT_FALSE(g3.contains(entities[i*2+1])); ASSERT_FALSE(g3.contains(entities[i*2])); - registry.assign(entities[i*2+1]); + registry.emplace(entities[i*2+1]); } ASSERT_EQ(g1.size(), 5u); @@ -892,7 +892,7 @@ TEST(Registry, NestedGroups) { ASSERT_TRUE(g2.contains(entities[i*2])); ASSERT_FALSE(g3.contains(entities[i*2+1])); ASSERT_FALSE(g3.contains(entities[i*2])); - registry.assign(entities[i*2]); + registry.emplace(entities[i*2]); } ASSERT_EQ(g1.size(), 5u); @@ -928,9 +928,9 @@ TEST(Registry, SortSingle) { int val = 0; - registry.assign(registry.create(), val++); - registry.assign(registry.create(), val++); - registry.assign(registry.create(), val++); + registry.emplace(registry.create(), val++); + registry.emplace(registry.create(), val++); + registry.emplace(registry.create(), val++); for(auto entity: registry.view()) { ASSERT_EQ(registry.get(entity), --val); @@ -951,8 +951,8 @@ TEST(Registry, SortMulti) { for(auto i = 0; i < 3; ++i) { const auto entity = registry.create(); - registry.assign(entity, uval++); - registry.assign(entity, ival++); + registry.emplace(entity, uval++); + registry.emplace(entity, ival++); } for(auto entity: registry.view()) { @@ -978,9 +978,9 @@ TEST(Registry, SortMulti) { TEST(Registry, SortEmpty) { entt::registry registry; - registry.assign(registry.create()); - registry.assign(registry.create()); - registry.assign(registry.create()); + registry.emplace(registry.create()); + registry.emplace(registry.create()); + registry.emplace(registry.create()); ASSERT_LT(registry.data()[0], registry.data()[1]); ASSERT_LT(registry.data()[1], registry.data()[2]); @@ -995,7 +995,7 @@ TEST(Registry, ComponentsWithTypesFromStandardTemplateLibrary) { // see #37 - the test shouldn't crash, that's all entt::registry registry; const auto entity = registry.create(); - registry.assign>(entity).insert(42); + registry.emplace>(entity).insert(42); registry.destroy(entity); } @@ -1003,7 +1003,7 @@ TEST(Registry, ConstructWithComponents) { // it should compile, that's all entt::registry registry; const auto value = 0; - registry.assign(registry.create(), value); + registry.emplace(registry.create(), value); } TEST(Registry, Signals) { @@ -1018,14 +1018,14 @@ TEST(Registry, Signals) { auto e0 = registry.create(); auto e1 = registry.create(); - registry.assign(e0); - registry.assign(e1); + registry.emplace(e0); + registry.emplace(e1); ASSERT_EQ(listener.counter, 2); ASSERT_EQ(listener.last, e1); - registry.assign(e1); - registry.assign(e0); + registry.emplace(e1); + registry.emplace(e0); ASSERT_EQ(listener.counter, 4); ASSERT_EQ(listener.last, e0); @@ -1048,8 +1048,8 @@ TEST(Registry, Signals) { registry.on_construct().disconnect<&listener::incr>(listener); registry.on_construct().disconnect<&listener::incr>(listener); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_EQ(listener.counter, 2); ASSERT_EQ(listener.last, e0); @@ -1057,7 +1057,7 @@ TEST(Registry, Signals) { registry.on_construct().connect<&listener::incr>(listener); registry.on_destroy().connect<&listener::decr>(listener); - registry.assign(e0); + registry.emplace(e0); registry.remove_if_exists(e1); ASSERT_EQ(listener.counter, 2); @@ -1067,7 +1067,7 @@ TEST(Registry, Signals) { registry.on_destroy().connect<&listener::decr>(listener); registry.remove_if_exists(e1); - registry.assign(e0); + registry.emplace(e0); ASSERT_EQ(listener.counter, 2); ASSERT_EQ(listener.last, e0); @@ -1078,10 +1078,10 @@ TEST(Registry, Signals) { ASSERT_EQ(listener.counter, 0); ASSERT_EQ(listener.last, e0); - registry.assign(e0); - registry.assign(e1); - registry.assign(e0); - registry.assign(e1); + registry.emplace(e0); + registry.emplace(e1); + registry.emplace(e0); + registry.emplace(e1); registry.destroy(e1); @@ -1090,8 +1090,8 @@ TEST(Registry, Signals) { registry.remove(e0); registry.remove(e0); - registry.assign_or_replace(e0); - registry.assign_or_replace(e0); + registry.emplace_or_replace(e0); + registry.emplace_or_replace(e0); ASSERT_EQ(listener.counter, 2); ASSERT_EQ(listener.last, e0); @@ -1099,17 +1099,17 @@ TEST(Registry, Signals) { registry.on_destroy().disconnect<&listener::decr>(listener); registry.on_destroy().disconnect<&listener::decr>(listener); - registry.assign_or_replace(e0); - registry.assign_or_replace(e0); + registry.emplace_or_replace(e0); + registry.emplace_or_replace(e0); ASSERT_EQ(listener.counter, 2); ASSERT_EQ(listener.last, e0); - registry.on_replace().connect<&listener::incr>(listener); - registry.on_replace().connect<&listener::incr>(listener); + registry.on_update().connect<&listener::incr>(listener); + registry.on_update().connect<&listener::incr>(listener); - registry.assign_or_replace(e0); - registry.assign_or_replace(e0); + registry.emplace_or_replace(e0); + registry.emplace_or_replace(e0); ASSERT_EQ(listener.counter, 4); ASSERT_EQ(listener.last, e0); @@ -1128,14 +1128,14 @@ TEST(Registry, RangeDestroy) { const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); - registry.assign(e2); + registry.emplace(e2); ASSERT_TRUE(registry.valid(e0)); ASSERT_TRUE(registry.valid(e1)); @@ -1160,28 +1160,28 @@ TEST(Registry, RangeDestroy) { ASSERT_FALSE(registry.valid(e2)); } -TEST(Registry, RangeAssign) { +TEST(Registry, RangeEmplace) { entt::registry registry; const auto e0 = registry.create(); const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); - registry.assign(e2); + registry.emplace(e2); ASSERT_FALSE(registry.has(e0)); ASSERT_FALSE(registry.has(e1)); ASSERT_FALSE(registry.has(e2)); const auto view = registry.view(); - registry.assign(view.begin(), view.end(), 3.f); + registry.insert(view.begin(), view.end(), 3.f); ASSERT_EQ(registry.get(e0), 3.f); ASSERT_EQ(registry.get(e1), 3.f); @@ -1189,7 +1189,7 @@ TEST(Registry, RangeAssign) { registry.clear(); float value[3]{0.f, 1.f, 2.f}; - registry.assign(registry.data(), registry.data() + registry.size(), value); + registry.insert(registry.data(), registry.data() + registry.size(), value, value + registry.size()); ASSERT_EQ(registry.get(e0), 0.f); ASSERT_EQ(registry.get(e1), 1.f); @@ -1203,14 +1203,14 @@ TEST(Registry, RangeRemove) { const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); - registry.assign(e2); + registry.emplace(e2); ASSERT_TRUE(registry.has(e0)); ASSERT_TRUE(registry.has(e1)); @@ -1229,14 +1229,14 @@ TEST(Registry, NonOwningGroupInterleaved) { typename entt::entity entity = entt::null; entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); const auto group = registry.group<>(entt::get); entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); std::size_t cnt{}; group.each([&cnt](auto...) { ++cnt; }); @@ -1249,14 +1249,14 @@ TEST(Registry, FullOwningGroupInterleaved) { typename entt::entity entity = entt::null; entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); const auto group = registry.group(); entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); std::size_t cnt{}; group.each([&cnt](auto...) { ++cnt; }); @@ -1269,14 +1269,14 @@ TEST(Registry, PartialOwningGroupInterleaved) { typename entt::entity entity = entt::null; entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); const auto group = registry.group(entt::get); entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); std::size_t cnt{}; group.each([&cnt](auto...) { ++cnt; }); @@ -1289,19 +1289,19 @@ TEST(Registry, NonOwningGroupSortInterleaved) { const auto group = registry.group<>(entt::get); const auto e0 = registry.create(); - registry.assign(e0, 0); - registry.assign(e0, '0'); + registry.emplace(e0, 0); + registry.emplace(e0, '0'); const auto e1 = registry.create(); - registry.assign(e1, 1); - registry.assign(e1, '1'); + registry.emplace(e1, 1); + registry.emplace(e1, '1'); registry.sort(std::greater{}); registry.sort(std::less{}); const auto e2 = registry.create(); - registry.assign(e2, 2); - registry.assign(e2, '2'); + registry.emplace(e2, 2); + registry.emplace(e2, '2'); group.each([e0, e1, e2](const auto entity, const auto &i, const auto &c) { if(entity == e0) { @@ -1317,10 +1317,10 @@ TEST(Registry, NonOwningGroupSortInterleaved) { }); } -TEST(Registry, GetOrAssign) { +TEST(Registry, GetOrEmplace) { entt::registry registry; const auto entity = registry.create(); - const auto value = registry.get_or_assign(entity, 3); + const auto value = registry.get_or_emplace(entity, 3); ASSERT_TRUE(registry.has(entity)); ASSERT_EQ(registry.get(entity), value); ASSERT_EQ(registry.get(entity), 3); @@ -1329,8 +1329,8 @@ TEST(Registry, GetOrAssign) { TEST(Registry, Constness) { entt::registry registry; - ASSERT_TRUE((std::is_same_v({})), int &>)); - ASSERT_TRUE((std::is_same_v({})), void>)); + ASSERT_TRUE((std::is_same_v({})), int &>)); + ASSERT_TRUE((std::is_same_v({})), void>)); ASSERT_TRUE((std::is_same_v({})), int &>)); ASSERT_TRUE((std::is_same_v({})), std::tuple>)); @@ -1352,19 +1352,19 @@ TEST(Registry, BatchCreateAmbiguousCall) { std::uint32_t foo = 32u; std::uint64_t bar = 64u; // this should work, no other tests required - registry.assign(entity, foo, bar); + registry.emplace(entity, foo, bar); } TEST(Registry, MoveOnlyComponent) { entt::registry registry; // the purpose is to ensure that move only types are always accepted - registry.assign>(registry.create()); + registry.emplace>(registry.create()); } TEST(Registry, NonDefaultConstructibleComponent) { entt::registry registry; // the purpose is to ensure that non default constructible type are always accepted - registry.assign(registry.create(), 42); + registry.emplace(registry.create(), 42); } TEST(Registry, Dependencies) { @@ -1372,17 +1372,17 @@ TEST(Registry, Dependencies) { const auto entity = registry.create(); // required because of an issue of VS2019 - constexpr auto assign_or_replace = &entt::registry::assign_or_replace; + constexpr auto emplace_or_replace = &entt::registry::emplace_or_replace; constexpr auto remove = &entt::registry::remove; - registry.on_construct().connect(); + registry.on_construct().connect(); registry.on_destroy().connect(); - registry.assign(entity, .3); + registry.emplace(entity, .3); ASSERT_FALSE(registry.has(entity)); ASSERT_EQ(registry.get(entity), .3); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE(registry.has(entity)); ASSERT_EQ(registry.get(entity), .0); @@ -1391,20 +1391,20 @@ TEST(Registry, Dependencies) { ASSERT_FALSE((registry.any(entity))); - registry.on_construct().disconnect(); + registry.on_construct().disconnect(); registry.on_destroy().disconnect(); - registry.assign(entity); + registry.emplace(entity); ASSERT_TRUE((registry.any(entity))); ASSERT_FALSE(registry.has(entity)); } -TEST(Registry, StableAssign) { +TEST(Registry, StableEmplace) { entt::registry registry; registry.on_construct().connect<&listener::sort>(); - registry.assign(registry.create(), 0); + registry.emplace(registry.create(), 0); - ASSERT_EQ(registry.assign(registry.create(), 1), 1); + ASSERT_EQ(registry.emplace(registry.create(), 1), 1); } TEST(Registry, AssignEntities) { @@ -1430,9 +1430,9 @@ TEST(Registry, Visit) { const auto entity = registry.create(); const auto other = registry.create(); - registry.assign(entity); - registry.assign(other); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(other); + registry.emplace(entity); bool hasType[3]{}; diff --git a/test/entt/entity/runtime_view.cpp b/test/entt/entity/runtime_view.cpp index 081fadb2e..01ead04b4 100644 --- a/test/entt/entity/runtime_view.cpp +++ b/test/entt/entity/runtime_view.cpp @@ -18,14 +18,14 @@ TEST(RuntimeView, Functionalities) { ASSERT_TRUE(view.empty()); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); ASSERT_FALSE(view.empty()); - registry.assign(e1); + registry.emplace(e1); auto it = view.begin(); @@ -52,8 +52,8 @@ TEST(RuntimeView, Iterator) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); ENTT_ID_TYPE types[] = { entt::type_info::id(), entt::type_info::id() }; auto view = registry.runtime_view(std::begin(types), std::end(types)); @@ -82,12 +82,12 @@ TEST(RuntimeView, Contains) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); registry.destroy(e0); @@ -102,13 +102,13 @@ TEST(RuntimeView, Empty) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ENTT_ID_TYPE types[] = { entt::type_info::id(), entt::type_info::id(), entt::type_info::id() }; auto view = registry.runtime_view(std::begin(types), std::end(types)); @@ -123,12 +123,12 @@ TEST(RuntimeView, Each) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ENTT_ID_TYPE types[] = { entt::type_info::id(), entt::type_info::id() }; auto view = registry.runtime_view(std::begin(types), std::end(types)); @@ -146,11 +146,11 @@ TEST(RuntimeView, EachWithHoles) { const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e0, '0'); - registry.assign(e1, '1'); + registry.emplace(e0, '0'); + registry.emplace(e1, '1'); - registry.assign(e0, 0); - registry.assign(e2, 2); + registry.emplace(e0, 0); + registry.emplace(e2, 2); ENTT_ID_TYPE types[] = { entt::type_info::id(), entt::type_info::id() }; auto view = registry.runtime_view(std::begin(types), std::end(types)); @@ -164,7 +164,7 @@ TEST(RuntimeView, MissingPool) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); ENTT_ID_TYPE types[] = { entt::type_info::id(), entt::type_info::id() }; auto view = registry.runtime_view(std::begin(types), std::end(types)); @@ -172,7 +172,7 @@ TEST(RuntimeView, MissingPool) { ASSERT_TRUE(view.empty()); ASSERT_EQ(view.size(), decltype(view.size()){0}); - registry.assign(e0); + registry.emplace(e0); ASSERT_TRUE(view.empty()); ASSERT_EQ(view.size(), decltype(view.size()){0}); @@ -187,7 +187,7 @@ TEST(RuntimeView, EmptyRange) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const ENTT_ID_TYPE *ptr = nullptr; auto view = registry.runtime_view(ptr, ptr); diff --git a/test/entt/entity/snapshot.cpp b/test/entt/entity/snapshot.cpp index cc67b4556..e09aa3208 100644 --- a/test/entt/entity/snapshot.cpp +++ b/test/entt/entity/snapshot.cpp @@ -68,18 +68,18 @@ TEST(Snapshot, Dump) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0, 42); - registry.assign(e0, 'c'); - registry.assign(e0, .1); + registry.emplace(e0, 42); + registry.emplace(e0, 'c'); + registry.emplace(e0, .1); const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e2, 3); + registry.emplace(e2, 3); const auto e3 = registry.create(); - registry.assign(e3); - registry.assign(e3, '0'); + registry.emplace(e3); + registry.emplace(e3, '0'); registry.destroy(e1); auto v1 = registry.current(e1); @@ -134,17 +134,17 @@ TEST(Snapshot, Partial) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0, 42); - registry.assign(e0, 'c'); - registry.assign(e0, .1); + registry.emplace(e0, 42); + registry.emplace(e0, 'c'); + registry.emplace(e0, .1); const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e2, 3); + registry.emplace(e2, 3); const auto e3 = registry.create(); - registry.assign(e3, '0'); + registry.emplace(e3, '0'); registry.destroy(e1); auto v1 = registry.current(e1); @@ -206,10 +206,10 @@ TEST(Snapshot, Iterator) { for(auto i = 0; i < 50; ++i) { const auto entity = registry.create(); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } @@ -271,13 +271,13 @@ TEST(Snapshot, Continuous) { entity = src.create(); entities.push_back(entity); - src.assign(entity); - src.assign(entity, i, i); + src.emplace(entity); + src.emplace(entity, i, i); if(i % 2) { - src.assign(entity, entity); + src.emplace(entity, entity); } else { - src.assign(entity); + src.emplace(entity); } } @@ -294,8 +294,8 @@ TEST(Snapshot, Continuous) { }); entity = dst.create(); - dst.assign(entity); - dst.assign(entity, -1, -1); + dst.emplace(entity); + dst.emplace(entity, -1, -1); entt::snapshot{src}.entities(output).component(output); @@ -525,10 +525,10 @@ TEST(Snapshot, SyncDataMembers) { auto child = src.create(); - src.assign(parent, entt::null); - src.assign(child, parent).quux.push_back(child); + src.emplace(parent, entt::null); + src.emplace(child, parent).quux.push_back(child); - src.assign( + src.emplace( child, decltype(map_component::keys){{{ child, 10 }}}, decltype(map_component::values){{{ 10, child }}}, diff --git a/test/entt/entity/view.cpp b/test/entt/entity/view.cpp index 2a2733976..9a0adc889 100644 --- a/test/entt/entity/view.cpp +++ b/test/entt/entity/view.cpp @@ -15,8 +15,8 @@ TEST(SingleComponentView, Functionalities) { ASSERT_TRUE(view.empty()); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); ASSERT_NO_THROW(registry.view().begin()++); ASSERT_NO_THROW(++registry.view().begin()); @@ -26,7 +26,7 @@ TEST(SingleComponentView, Functionalities) { ASSERT_EQ(view.size(), typename decltype(view)::size_type{1}); ASSERT_FALSE(view.empty()); - registry.assign(e0); + registry.emplace(e0); ASSERT_EQ(view.size(), typename decltype(view)::size_type{2}); @@ -56,10 +56,10 @@ TEST(SingleComponentView, ElementAccess) { auto cview = std::as_const(registry).view(); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); for(typename decltype(view)::size_type i{}; i < view.size(); ++i) { ASSERT_EQ(view[i], i ? e0 : e1); @@ -71,10 +71,10 @@ TEST(SingleComponentView, Contains) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); registry.destroy(e0); @@ -88,11 +88,11 @@ TEST(SingleComponentView, Empty) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); auto view = registry.view(); @@ -103,8 +103,8 @@ TEST(SingleComponentView, Empty) { TEST(SingleComponentView, Each) { entt::registry registry; - registry.assign(registry.create()); - registry.assign(registry.create()); + registry.emplace(registry.create()); + registry.emplace(registry.create()); auto view = registry.view(); std::size_t cnt = 0; @@ -128,7 +128,7 @@ TEST(SingleComponentView, ConstNonConstAndAllInBetween) { ASSERT_EQ(view.size(), decltype(view.size()){0}); ASSERT_EQ(cview.size(), decltype(cview.size()){0}); - registry.assign(registry.create(), 0); + registry.emplace(registry.create(), 0); ASSERT_EQ(view.size(), decltype(view.size()){1}); ASSERT_EQ(cview.size(), decltype(cview.size()){1}); @@ -155,16 +155,16 @@ TEST(SingleComponentView, Find) { auto view = registry.view(); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); const auto e2 = registry.create(); - registry.assign(e2); + registry.emplace(e2); const auto e3 = registry.create(); - registry.assign(e3); + registry.emplace(e3); registry.remove(e1); @@ -184,7 +184,7 @@ TEST(SingleComponentView, Find) { const auto e4 = registry.create(); registry.destroy(e4); const auto e5 = registry.create(); - registry.assign(e5); + registry.emplace(e5); ASSERT_NE(view.find(e5), view.end()); ASSERT_EQ(view.find(e4), view.end()); @@ -194,7 +194,7 @@ TEST(SingleComponentView, Less) { entt::registry registry; auto create = [&](auto... component) { const auto entt = registry.create(); - (registry.assign(entt, component), ...); + (registry.emplace(entt, component), ...); return entt; }; @@ -228,10 +228,10 @@ TEST(SingleComponentView, FrontBack) { ASSERT_EQ(view.back(), static_cast(entt::null)); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); ASSERT_EQ(view.front(), e1); ASSERT_EQ(view.back(), e0); @@ -247,16 +247,16 @@ TEST(MultiComponentView, Functionalities) { ASSERT_TRUE((cview.empty())); const auto e0 = registry.create(); - registry.assign(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); + registry.emplace(e1); ASSERT_FALSE(view.empty()); ASSERT_FALSE((view.empty())); ASSERT_FALSE((cview.empty())); - registry.assign(e1); + registry.emplace(e1); auto it = registry.view().begin(); @@ -294,8 +294,8 @@ TEST(MultiComponentView, Functionalities) { TEST(MultiComponentView, Iterator) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); + registry.emplace(entity); + registry.emplace(entity); const auto view = registry.view(); using iterator = typename decltype(view)::iterator; @@ -323,12 +323,12 @@ TEST(MultiComponentView, Contains) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); registry.destroy(e0); @@ -342,13 +342,13 @@ TEST(MultiComponentView, Empty) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); auto view = registry.view(); @@ -360,12 +360,12 @@ TEST(MultiComponentView, Each) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); auto view = registry.view(); auto cview = std::as_const(registry).view(); @@ -387,13 +387,13 @@ TEST(MultiComponentView, EachWithType) { for(auto i = 0; i < 3; ++i) { const auto entity = registry.create(); - registry.assign(entity, i); - registry.assign(entity); + registry.emplace(entity, i); + registry.emplace(entity); } // makes char a better candidate during iterations const auto entity = registry.create(); - registry.assign(entity, 99); + registry.emplace(entity, 99); registry.view().each([value = 2](const auto curr, const auto) mutable { ASSERT_EQ(curr, value--); @@ -415,11 +415,11 @@ TEST(MultiComponentView, EachWithHoles) { const auto e1 = registry.create(); const auto e2 = registry.create(); - registry.assign(e0, '0'); - registry.assign(e1, '1'); + registry.emplace(e0, '0'); + registry.emplace(e1, '1'); - registry.assign(e0, 0); - registry.assign(e2, 2); + registry.emplace(e0, 0); + registry.emplace(e2, 2); auto view = registry.view(); @@ -437,8 +437,8 @@ TEST(MultiComponentView, ConstNonConstAndAllInBetween) { ASSERT_EQ(view.size(), decltype(view.size()){0}); const auto entity = registry.create(); - registry.assign(entity, 0); - registry.assign(entity, 'c'); + registry.emplace(entity, 0); + registry.emplace(entity, 'c'); ASSERT_EQ(view.size(), decltype(view.size()){1}); @@ -459,20 +459,20 @@ TEST(MultiComponentView, Find) { auto view = registry.view(); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); const auto e2 = registry.create(); - registry.assign(e2); - registry.assign(e2); + registry.emplace(e2); + registry.emplace(e2); const auto e3 = registry.create(); - registry.assign(e3); - registry.assign(e3); + registry.emplace(e3); + registry.emplace(e3); registry.remove(e1); @@ -492,8 +492,8 @@ TEST(MultiComponentView, Find) { const auto e4 = registry.create(); registry.destroy(e4); const auto e5 = registry.create(); - registry.assign(e5); - registry.assign(e5); + registry.emplace(e5); + registry.emplace(e5); ASSERT_NE(view.find(e5), view.end()); ASSERT_EQ(view.find(e4), view.end()); @@ -503,20 +503,20 @@ TEST(MultiComponentView, ExcludedComponents) { entt::registry registry; const auto e0 = registry.create(); - registry.assign(e0, 0); + registry.emplace(e0, 0); const auto e1 = registry.create(); - registry.assign(e1, 1); - registry.assign(e1); + registry.emplace(e1, 1); + registry.emplace(e1); const auto view = registry.view(entt::exclude); const auto e2 = registry.create(); - registry.assign(e2, 2); + registry.emplace(e2, 2); const auto e3 = registry.create(); - registry.assign(e3, 3); - registry.assign(e3); + registry.emplace(e3, 3); + registry.emplace(e3); for(const auto entity: view) { ASSERT_TRUE(entity == e0 || entity == e2); @@ -528,8 +528,8 @@ TEST(MultiComponentView, ExcludedComponents) { } } - registry.assign(e0); - registry.assign(e2); + registry.emplace(e0); + registry.emplace(e2); registry.remove(e1); registry.remove(e3); @@ -548,14 +548,14 @@ TEST(MultiComponentView, Less) { entt::registry registry; const auto entity = registry.create(); - registry.assign(entity); - registry.assign(entity); - registry.assign(entity); - registry.assign>(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace(entity); + registry.emplace>(entity); const auto other = registry.create(); - registry.assign(other); - registry.assign(other); + registry.emplace(other); + registry.emplace(other); registry.view>().less([entity](const auto entt, int, char) { ASSERT_EQ(entity, entt); @@ -592,15 +592,15 @@ TEST(MultiComponentView, FrontBack) { ASSERT_EQ(view.back(), static_cast(entt::null)); const auto e0 = registry.create(); - registry.assign(e0); - registry.assign(e0); + registry.emplace(e0); + registry.emplace(e0); const auto e1 = registry.create(); - registry.assign(e1); - registry.assign(e1); + registry.emplace(e1); + registry.emplace(e1); const auto entity = registry.create(); - registry.assign(entity); + registry.emplace(entity); ASSERT_EQ(view.front(), e1); ASSERT_EQ(view.back(), e0); diff --git a/test/lib/registry/lib.cpp b/test/lib/registry/lib.cpp index 581f34eec..09ee3481b 100644 --- a/test/lib/registry/lib.cpp +++ b/test/lib/registry/lib.cpp @@ -9,11 +9,11 @@ ENTT_API void update_position(entt::registry ®istry) { }); } -ENTT_API void assign_velocity(entt::registry ®istry) { +ENTT_API void emplace_velocity(entt::registry ®istry) { // forces the creation of the pool for the velocity component registry.prepare(); for(auto entity: registry.view()) { - registry.assign(entity, 1., 1.); + registry.emplace(entity, 1., 1.); } } diff --git a/test/lib/registry/main.cpp b/test/lib/registry/main.cpp index 493a20551..844739d60 100644 --- a/test/lib/registry/main.cpp +++ b/test/lib/registry/main.cpp @@ -5,17 +5,17 @@ #include "types.h" ENTT_API void update_position(entt::registry &); -ENTT_API void assign_velocity(entt::registry &); +ENTT_API void emplace_velocity(entt::registry &); TEST(Lib, Registry) { entt::registry registry; for(auto i = 0; i < 3; ++i) { const auto entity = registry.create(); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); } - assign_velocity(registry); + emplace_velocity(registry); update_position(registry); ASSERT_EQ(registry.size(), registry.size()); diff --git a/test/lib/registry_plugin/main.cpp b/test/lib/registry_plugin/main.cpp index 0ff70710b..31baedc38 100644 --- a/test/lib/registry_plugin/main.cpp +++ b/test/lib/registry_plugin/main.cpp @@ -14,10 +14,10 @@ TEST(Lib, Registry) { for(auto i = 0; i < 3; ++i) { const auto entity = registry.create(); - registry.assign(entity, i, i); + registry.emplace(entity, i, i); if(i % 2) { - registry.assign(entity); + registry.emplace(entity); } } diff --git a/test/lib/registry_plugin/plugin.cpp b/test/lib/registry_plugin/plugin.cpp index 8e7c7eb90..330064a40 100644 --- a/test/lib/registry_plugin/plugin.cpp +++ b/test/lib/registry_plugin/plugin.cpp @@ -13,7 +13,7 @@ CR_EXPORT int cr_main(cr_plugin *ctx, cr_op operation) { auto ®istry = *static_cast(ctx->userdata); const auto position_view = registry.view(entt::exclude); - registry.assign(position_view.begin(), position_view.end(), velocity{1., 1.}); + registry.insert(position_view.begin(), position_view.end(), velocity{1., 1.}); registry.view().each([](auto &pos, auto &vel) { pos.x += static_cast(16 * vel.dx); diff --git a/test/snapshot/snapshot.cpp b/test/snapshot/snapshot.cpp index f5d42ac68..b1fadb6b2 100644 --- a/test/snapshot/snapshot.cpp +++ b/test/snapshot/snapshot.cpp @@ -42,19 +42,19 @@ TEST(Snapshot, Full) { entt::registry destination; auto e0 = source.create(); - source.assign(e0, 16.f, 16.f); + source.emplace(e0, 16.f, 16.f); source.destroy(source.create()); auto e1 = source.create(); - source.assign(e1, .8f, .0f); - source.assign(e1, e0); + source.emplace(e1, .8f, .0f); + source.emplace(e1, e0); auto e2 = source.create(); auto e3 = source.create(); - source.assign(e3, 1000, 100); - source.assign>(e3); + source.emplace(e3, 1000, 100); + source.emplace>(e3); source.destroy(e2); auto v2 = source.current(e2); @@ -106,21 +106,21 @@ TEST(Snapshot, Continuous) { } auto e0 = source.create(); - source.assign(e0, 0.f, 0.f); - source.assign(e0, e0); + source.emplace(e0, 0.f, 0.f); + source.emplace(e0, e0); auto e1 = source.create(); - source.assign(e1, 1.f, 1.f); - source.assign(e1, e0); + source.emplace(e1, 1.f, 1.f); + source.emplace(e1, e0); auto e2 = source.create(); - source.assign(e2, .2f, .2f); - source.assign(e2, e0); + source.emplace(e2, .2f, .2f); + source.emplace(e2, e0); auto e3 = source.create(); - source.assign(e3, 1000, 1000); - source.assign(e3, e2); - source.assign>(e3); + source.emplace(e3, 1000, 1000); + source.emplace(e3, e2); + source.emplace>(e3); { // output finishes flushing its contents when it goes out of scope