s/DetectChanges/DetectChangesMut/ and doing the SAFETY comment the 'right' way

Author: Guvante

crates/bevy_ecs/src/change_detection.rs

@@ -241,7 +241,7 @@ macro_rules! impl_methods {
             #[doc = stringify!($name)]
             /// <T>`, but you need a `Mut<T>`.
             ///
-            /// Note that calling [`DetectChanges::set_last_changed`] on the returned value
+            /// Note that calling [`DetectChangesMut::set_last_changed`] on the returned value
             /// will not affect the original.
             pub fn reborrow(&mut self) -> Mut<'_, $target> {
                 Mut {
@@ -606,7 +606,7 @@ impl<'a> MutUntyped<'a> {
     /// Returns a [`MutUntyped`] with a smaller lifetime.
     /// This is useful if you have `&mut MutUntyped`, but you need a `MutUntyped`.
     ///
-    /// Note that calling [`DetectChanges::set_last_changed`] on the returned value
+    /// Note that calling [`DetectChangesMut::set_last_changed`] on the returned value
     /// will not affect the original.
     #[inline]
     pub fn reborrow(&mut self) -> MutUntyped {

crates/bevy_ecs/src/world/mod.rs

@@ -1589,12 +1589,11 @@ impl World {
         let change_tick = self.change_tick();
         let (ptr, ticks) = self.get_resource_with_ticks(component_id)?;
 
-        let ticks = unsafe {
+        let ticks =
             // SAFETY: This function has exclusive access to the world so nothing aliases `ticks`.
             // - index is in-bounds because the column is initialized and non-empty
             // - no other reference to the ticks of the same row can exist at the same time
-            TicksMut::from_tick_cells(ticks, self.last_change_tick(), change_tick)
-        };
+            unsafe { TicksMut::from_tick_cells(ticks, self.last_change_tick(), change_tick) };
 
         Some(MutUntyped {
             // SAFETY: This function has exclusive access to the world so nothing aliases `ptr`.
@@ -1634,12 +1633,11 @@ impl World {
         let change_tick = self.change_tick();
         let (ptr, ticks) = self.get_non_send_with_ticks(component_id)?;
 
-        let ticks = unsafe {
+        let ticks =
             // SAFETY: This function has exclusive access to the world so nothing aliases `ticks`.
             // - index is in-bounds because the column is initialized and non-empty
             // - no other reference to the ticks of the same row can exist at the same time
-            TicksMut::from_tick_cells(ticks, self.last_change_tick(), change_tick)
-        };
+            unsafe { TicksMut::from_tick_cells(ticks, self.last_change_tick(), change_tick) };
 
         Some(MutUntyped {
             // SAFETY: This function has exclusive access to the world so nothing aliases `ptr`.