remove unneccessary safety requirement

Author: jakobhellermann

crates/bevy_ecs/src/storage/mod.rs

@@ -30,7 +30,6 @@ impl Storages {
     ///
     /// # Safety
     /// - `entity_location` must be within bounds of the given archetype and `entity` must exist inside
-    /// - `component_id` must be valid
     /// - `storage_type` must accurately reflect where the components for `component_id` are stored.
     /// - `Archetypes` and `Components` must come from the world this of this `Storages`
     /// - the caller must ensure that no aliasing rules are violated
@@ -45,6 +44,7 @@ impl Storages {
         location: EntityLocation,
     ) -> Option<(Ptr<'_>, TickCells<'_>)> {
         let component_id = components.get_id(type_id)?;
+        // SAFETY: component_id is valid, the rest is deferred to caller
         self.get_component_and_ticks(archetypes, component_id, storage_type, entity, location)
     }
 
@@ -88,6 +88,7 @@ impl Storages {
     /// # Safety
     /// - `entity_location` must be within bounds of the given archetype and `entity` must exist inside
     /// the archetype
+    /// - `storage_type` must accurately reflect where the components for `component_id` are stored.
     /// - `Archetypes` and `Components` must come from the world this of this `Storages`
     /// - the caller must ensure that no aliasing rules are violated
     #[inline]
@@ -101,6 +102,7 @@ impl Storages {
         location: EntityLocation,
     ) -> Option<Ptr<'_>> {
         let component_id = components.get_id(type_id)?;
+        // SAFETY: component_id is valid, the rest is deferred to caller
         self.get_component(archetypes, component_id, storage_type, entity, location)
     }
 
@@ -139,6 +141,7 @@ impl Storages {
     /// # Safety
     /// - `entity_location` must be within bounds of the given archetype and `entity` must exist inside
     /// the archetype
+    /// - `storage_type` must accurately reflect where the components for `component_id` are stored.
     /// - `Archetypes` and `Components` must come from the world this of this `Storages`
     /// - the caller must ensure that no aliasing rules are violated
     #[inline]
@@ -152,6 +155,7 @@ impl Storages {
         location: EntityLocation,
     ) -> Option<ComponentTicks> {
         let component_id = components.get_id(type_id)?;
+        // SAFETY: component_id is valid, the rest is deferred to caller
         self.get_ticks(archetypes, component_id, storage_type, entity, location)
     }