-
-
Notifications
You must be signed in to change notification settings - Fork 4k
Closed
Labels
A-ECSEntities, components, systems, and eventsEntities, components, systems, and eventsD-ComplexQuite challenging from either a design or technical perspective. Ask for help!Quite challenging from either a design or technical perspective. Ask for help!S-Needs-ReviewNeeds reviewer attention (from anyone!) to move forwardNeeds reviewer attention (from anyone!) to move forwardX-ContentiousThere are nontrivial implications that should be thought throughThere are nontrivial implications that should be thought through
Milestone
Description
I realized that because
Storages
andComponents
aren't public fields onWorld
, and there are no mutable accessor functions, doing the same change likely doesn't work forinit_state
.
Adding mutable accessor function does not make sense, because this would allow users to readily cause UB.
Instead we could have a function on&mut World
that returns an opaque struct on whichinit_component
, ... can be called.
Originally posted by @Victoronz in #13343 (comment)
Metadata
Metadata
Assignees
Labels
A-ECSEntities, components, systems, and eventsEntities, components, systems, and eventsD-ComplexQuite challenging from either a design or technical perspective. Ask for help!Quite challenging from either a design or technical perspective. Ask for help!S-Needs-ReviewNeeds reviewer attention (from anyone!) to move forwardNeeds reviewer attention (from anyone!) to move forwardX-ContentiousThere are nontrivial implications that should be thought throughThere are nontrivial implications that should be thought through