-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Open
Labels
A-ECSEntities, components, systems, and eventsEntities, components, systems, and eventsC-PerformanceA change motivated by improving speed, memory usage or compile timesA change motivated by improving speed, memory usage or compile timesS-Needs-BenchmarkingThis set of changes needs performance benchmarking to double-check that they helpThis set of changes needs performance benchmarking to double-check that they help
Description
What problem does this solve or what need does it fill?
While working on #18058, I realized I could further optimize the code if could stop relationship components from removing the relation ship target component when it's empty. Otherwise removing relationship components may lead to archetype moves which get undone when replace_related finishes
What solution would you like?
- Rename
RelationshipInsertHookModetoRelationShipHookMode - Add
remove_with_relationship_insert_hook_modeandtake_with_relationship_insert_hook_modeanalogous toinsert_with_relationship_insert_hook_modebut for single components.
What alternative(s) have you considered?
Just paying for the archetype moves. The performance hit probably isn't going to be too noticeable.
Metadata
Metadata
Assignees
Labels
A-ECSEntities, components, systems, and eventsEntities, components, systems, and eventsC-PerformanceA change motivated by improving speed, memory usage or compile timesA change motivated by improving speed, memory usage or compile timesS-Needs-BenchmarkingThis set of changes needs performance benchmarking to double-check that they helpThis set of changes needs performance benchmarking to double-check that they help