diff --git a/library/core/src/iter/adapters/step_by.rs b/library/core/src/iter/adapters/step_by.rs index 2458a13d987b5..7f58f7d17758d 100644 --- a/library/core/src/iter/adapters/step_by.rs +++ b/library/core/src/iter/adapters/step_by.rs @@ -141,7 +141,7 @@ impl SpecRangeSetup for T { /// Specialization trait to optimize `StepBy>` iteration. /// -/// # Correctness +/// # Safety /// /// Technically this is safe to implement (look ma, no unsafe!), but in reality /// a lot of unsafe code relies on ranges over integers being correct. @@ -149,7 +149,7 @@ impl SpecRangeSetup for T { /// For correctness *all* public StepBy methods must be specialized /// because `setup` drastically alters the meaning of the struct fields so that mixing /// different implementations would lead to incorrect results. -trait StepByImpl { +unsafe trait StepByImpl { type Item; fn spec_next(&mut self) -> Option; @@ -172,13 +172,13 @@ trait StepByImpl { /// /// See also: `StepByImpl` /// -/// # Correctness +/// # Safety /// /// The specializations must be implemented together with `StepByImpl` /// where applicable. I.e. if `StepBy` does support backwards iteration /// for a given iterator and that is specialized for forward iteration then /// it must also be specialized for backwards iteration. -trait StepByBackImpl { +unsafe trait StepByBackImpl { type Item; fn spec_next_back(&mut self) -> Option @@ -201,7 +201,7 @@ trait StepByBackImpl { F: FnMut(Acc, Self::Item) -> Acc; } -impl StepByImpl for StepBy { +unsafe impl StepByImpl for StepBy { type Item = I::Item; #[inline] @@ -319,7 +319,7 @@ impl StepByImpl for StepBy { } } -impl StepByBackImpl for StepBy { +unsafe impl StepByBackImpl for StepBy { type Item = I::Item; #[inline] @@ -416,7 +416,7 @@ macro_rules! spec_int_ranges { } } - impl StepByImpl> for StepBy> { + unsafe impl StepByImpl> for StepBy> { #[inline] fn spec_next(&mut self) -> Option<$t> { // if a step size larger than the type has been specified fall back to @@ -497,7 +497,7 @@ macro_rules! spec_int_ranges_r { ($($t:ty)*) => ($( const _: () = assert!(usize::BITS >= <$t>::BITS); - impl StepByBackImpl> for StepBy> { + unsafe impl StepByBackImpl> for StepBy> { #[inline] fn spec_next_back(&mut self) -> Option