use mem::swap instead of ptr::swap_nonoverlapping

RalfJung · RalfJung · commit e7a1e4271d3c · 2023-08-16T08:47:06.000+02:00
diff --git a/library/core/src/cell.rs b/library/core/src/cell.rs
@@ -450,7 +450,7 @@ impl<T> Cell<T> {
         // either of these `Cell`s. We also excluded shenanigans like partially overlapping `Cell`s,
         // so `swap` will just properly copy two full values of type `T` back and forth.
         unsafe {
-            ptr::swap_nonoverlapping(self.value.get(), other.value.get(), 1);
+            mem::swap(&mut *self.value.get(), &mut *other.value.get());
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ impl<T> Cell<T> {`
`450`	`450`	// either of these `Cell`s. We also excluded shenanigans like partially overlapping `Cell`s,
`451`	`451`	// so `swap` will just properly copy two full values of type `T` back and forth.
`452`	`452`	`unsafe {`
`453`		`- ptr::swap_nonoverlapping(self.value.get(), other.value.get(), 1);`
	`453`	`+ mem::swap(&mut self.value.get(), &mut other.value.get());`
`454`	`454`	`}`
`455`	`455`	`}`
`456`	`456`