tweak safety comments

Author: jakobhellermann

crates/bevy_ecs/src/reflect.rs

@@ -52,7 +52,7 @@ pub struct ReflectComponentFns {
     /// Function pointer implementing [`ReflectComponent::reflect()`].
     pub reflect: fn(&World, Entity) -> Option<&dyn Reflect>,
     /// Function pointer implementing [`ReflectComponent::reflect_mut()`].
-    /// The function may only be called with a InteriorMutableWorld that can be used to access the relevant component on the given entity
+    /// SAFETY:  The function may only be called with an InteriorMutableWorld that can be used to mutably access the relevant component on the given entity
     pub reflect_mut: unsafe fn(InteriorMutableWorld<'_>, Entity) -> Option<Mut<'_, dyn Reflect>>,
     /// Function pointer implementing [`ReflectComponent::copy()`].
     pub copy: fn(&World, &mut World, Entity, Entity),
@@ -124,7 +124,7 @@ impl ReflectComponent {
     /// # Safety
     /// This method does not prevent you from having two mutable pointers to the same data,
     /// violating Rust's aliasing rules. To avoid this:
-    /// * Only call this method with a [`InteriorMutableWorld`] that may be used to access the component on the entity `entity`
+    /// * Only call this method with a [`InteriorMutableWorld`] that may be used to mutably access the component on the entity `entity`
     /// * Don't call this method more than once in the same scope for a given [`Component`].
     pub unsafe fn reflect_unchecked_mut<'a>(
         &self,
@@ -264,6 +264,7 @@ pub struct ReflectResourceFns {
     /// Function pointer implementing [`ReflectResource::reflect()`].
     pub reflect: fn(&World) -> Option<&dyn Reflect>,
     /// Function pointer implementing [`ReflectResource::reflect_unchecked_mut()`].
+    /// SAFETY: The function may only be called with an InteriorMutableWorld that can be used to mutably access the relevant resource
     pub reflect_unchecked_mut: unsafe fn(InteriorMutableWorld<'_>) -> Option<Mut<'_, dyn Reflect>>,
     /// Function pointer implementing [`ReflectResource::copy()`].
     pub copy: fn(&World, &mut World),
@@ -319,7 +320,7 @@ impl ReflectResource {
     /// # Safety
     /// This method does not prevent you from having two mutable pointers to the same data,
     /// violating Rust's aliasing rules. To avoid this:
-    /// * Only call this method with a [`InteriorMutableWorld`] which can be used to access the resource.
+    /// * Only call this method with a [`InteriorMutableWorld`] which can be used to mutably access the resource.
     /// * Don't call this method more than once in the same scope for a given [`Resource`].
     pub unsafe fn reflect_unchecked_mut<'w>(
         &self,

crates/bevy_ecs/src/world/entity_ref.rs

@@ -142,7 +142,7 @@ impl<'w> EntityRef<'w> {
         let info = self.world.components().get_info(component_id)?;
         // SAFETY:
         // - entity_location and entity are valid
-        // . component_id is valid as checked by the line above
+        // - component_id is valid as checked by the line above
         // - the storage type is accurate as checked by the fetched ComponentInfo
         unsafe {
             self.world.get_component(
@@ -790,7 +790,7 @@ pub(crate) unsafe fn get_mut<T: Component>(
     // SAFETY:
     // - world access is unique
     // - entity location is valid
-    // - and returned component is of type T
+    // - returned component is of type T
     world
         .get_component_and_ticks_with_type(
             TypeId::of::<T>(),
@@ -821,7 +821,7 @@ pub(crate) unsafe fn get_mut_by_id(
     // SAFETY:
     // - world access is unique
     // - entity location is valid
-    // - and returned component is of type T
+    // - returned component is of type T
     world
         .get_component_and_ticks(component_id, info.storage_type(), entity, location)
         .map(|(value, ticks)| MutUntyped {