Changeset 25251

Timestamp:

Apr 13, 2021, 3:47:59 PM (3 years ago)

Author:

wraitii

Message:

Destroy components in reverse order of construction.

Components have some initialisation-order dependency. They could also have destroy-order dependency, and from a RAII-like point of view, it makes sense to do this.

Differential Revision: ​https://code.wildfiregames.com/D3843

File:

• ps/trunk/source/simulation2/system/ComponentManager.cpp (modified) (1 diff)

ps/trunk/source/simulation2/system/ComponentManager.cpp

@@ -460 +460 @@
     m_DynamicMessageSubscriptionsNonsyncByComponent.clear();
 
-    // Delete all IComponents
-    std::map<ComponentTypeId, std::map<entity_id_t, IComponent*> >::iterator iit = m_ComponentsByTypeId.begin();
-    for (; iit != m_ComponentsByTypeId.end(); ++iit)
+    // Delete all IComponents
+    std::map<ComponentTypeId, std::map<entity_id_t, IComponent*> >::begin();
+    for (; iit != m_ComponentsByTypeId.end(); ++iit)
     {
         std::map<entity_id_t, IComponent*>::iterator eit = iit->second.begin();