From a13e7212025e5339133126d129e64fe8e7aff84b Mon Sep 17 00:00:00 2001 From: Steven Allen Date: Wed, 21 Jan 2015 00:14:04 -0500 Subject: [PATCH] Use the default `len` implementation in ExactSizeIterator impls --- src/libcore/iter.rs | 29 +++++------------------------ 1 file changed, 5 insertions(+), 24 deletions(-) diff --git a/src/libcore/iter.rs b/src/libcore/iter.rs index cfbccef2a72..e39c60a4006 100644 --- a/src/libcore/iter.rs +++ b/src/libcore/iter.rs @@ -1088,10 +1088,7 @@ fn next_back(&mut self) -> Option<::Item> { self.iter.next_back() } #[stable] -impl<'a, I> ExactSizeIterator for ByRef<'a, I> where I: 'a + ExactSizeIterator { - #[inline] - fn len(&self) -> uint { self.iter.len() } -} +impl<'a, I> ExactSizeIterator for ByRef<'a, I> where I: 'a + ExactSizeIterator {} /// A trait for iterators over elements which can be added together #[unstable = "needs to be re-evaluated as part of numerics reform"] @@ -1797,14 +1794,7 @@ fn size_hint(&self) -> (uint, Option) { } #[stable] -impl ExactSizeIterator for Peekable where I: ExactSizeIterator { - #[inline] - fn len(&self) -> usize { - // This is guarenteed to not overflow because `len()` must have been able to return a valid - // value before we peeked. - self.iter.len() + if self.peeked.is_some() { 1 } else { 0 } - } -} +impl ExactSizeIterator for Peekable where I: ExactSizeIterator {} #[stable] impl Peekable where I: Iterator { @@ -1999,10 +1989,7 @@ fn idx(&mut self, index: uint) -> Option<::Item> { } #[stable] -impl ExactSizeIterator for Skip where I: ExactSizeIterator { - #[inline] - fn len(&self) -> uint { self.iter.len().saturating_sub(self.n) } -} +impl ExactSizeIterator for Skip where I: ExactSizeIterator {} /// An iterator that only iterates over the first `n` iterations of `iter`. #[derive(Clone)] @@ -2060,10 +2047,7 @@ fn idx(&mut self, index: uint) -> Option<::Item> { } #[stable] -impl ExactSizeIterator for Take where I: ExactSizeIterator { - #[inline] - fn len(&self) -> uint { cmp::min(self.iter.len(), self.n) } -} +impl ExactSizeIterator for Take where I: ExactSizeIterator {} /// An iterator to maintain state while iterating another iterator @@ -2275,10 +2259,7 @@ fn idx(&mut self, index: uint) -> Option<::Item> { } #[stable] -impl ExactSizeIterator for Fuse where I: ExactSizeIterator { - #[inline] - fn len(&self) -> uint { self.iter.len() } -} +impl ExactSizeIterator for Fuse where I: ExactSizeIterator {} impl Fuse { /// Resets the fuse such that the next call to .next() or .next_back() will -- 2.44.0