Skip to content

Comments

[Core]: Implement entities manager#53

Merged
nircoe merged 5 commits intorefactor/dod_ecsfrom
nircoe/refactor/dod_ecs/entities
Feb 14, 2026
Merged

[Core]: Implement entities manager#53
nircoe merged 5 commits intorefactor/dod_ecsfrom
nircoe/refactor/dod_ecs/entities

Conversation

@nircoe
Copy link
Owner

@nircoe nircoe commented Feb 14, 2026

  • Implement entities central manager with type-indexed component_pool storage
  • Refactor component_pool to inherit basic_component_pool abstract base for type erasure
  • Add erase_at(dense_index) to sparse_set for index-based removal (eliminates double lookup in remove())
  • Change component_pool::get() to return T& with assert (fail-fast over std::optional)
  • Remove entity::valid() (liveness check belongs to entities::valid())
  • Add EntitiesTests suite covering lifecycle, components, for_each, and bulk performance
  • Consolidate integration tests into entities_tests.cpp, remove redundant test files

@nircoe nircoe self-assigned this Feb 14, 2026
@nircoe nircoe marked this pull request as ready for review February 14, 2026 15:57
@nircoe nircoe merged commit 32fd1dc into refactor/dod_ecs Feb 14, 2026
5 checks passed
@nircoe nircoe deleted the nircoe/refactor/dod_ecs/entities branch February 14, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant