let observers escape to heap

Author: mlange-42

ecs/events.go

@@ -122,9 +122,9 @@ type observerManager struct {
 }
 
 // newObserverManager creates anew empty observerManager.
-func newObserverManager() observerManager {
+func newObserverManager() *observerManager {
 	maxEvents := math.MaxUint8 + 1
-	return observerManager{
+	return &observerManager{
 		observers:    make([][]*observerData, maxEvents),
 		hasObservers: make([]bool, maxEvents),
 		anyNoComps:   make([]bool, maxEvents),

ecs/storage.go

@@ -13,7 +13,6 @@ type storage struct {
 	entities           []entityIndex      // Entity positions in archetypes, indexed by entity ID
 	isTarget           []bool             // Whether each entity is a target of a relationship
 	graph              graph              // Graph for fast archetype traversal
-	slices             *slices            // Slices for internal re-use
 	archetypes         []archetype        // All archetypes
 	allArchetypes      []archetypeID      // list of all archetype IDs to simplify usage of componentIndex
 	componentIndex     [][]archetypeID    // Archetypes indexed by components IDs; each archetype appears under all its component IDs
@@ -24,8 +23,9 @@ type storage struct {
 	entityPool         entityPool         // Entity pool for creation and recycling
 	registry           componentRegistry  // Component registry
 	locks              lock               // World locks
-	observers          observerManager    // Observer/event manager
 	config             config             // Storage configuration (initial capacities)
+	slices             *slices            // Slices for internal re-use
+	observers          *observerManager   // Observer/event manager
 }
 
 // componentStorage is an index for faster access of table columns by component ID.

ecs/world_test.go

@@ -488,7 +488,7 @@ func TestWorldReset(t *testing.T) {
 	expectEqual(t, 4, query.Count())
 	query.Close()
 
-	obs := &world.storage.observers
+	obs := world.storage.observers
 	expectEqual(t, maxObserverID, o.id)
 	expectTrue(t, obs.allComps[OnCreateEntity].IsZero())
 	expectTrue(t, obs.allWith[OnCreateEntity].IsZero())