Skip to content

Commit e5a4211

Browse files
committed
Fixes clippy complaining about return None instead of question marks
1 parent 790ab24 commit e5a4211

File tree

2 files changed

+2
-9
lines changed

2 files changed

+2
-9
lines changed

src/impl_methods.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2299,10 +2299,7 @@ where
22992299
let dim = dim.into_dimension();
23002300

23012301
// Note: zero strides are safe precisely because we return an read-only view
2302-
let broadcast_strides = match upcast(&dim, &self.dim, &self.strides) {
2303-
Some(st) => st,
2304-
None => return None,
2305-
};
2302+
let broadcast_strides = upcast(&dim, &self.dim, &self.strides)?;
23062303
unsafe { Some(ArrayView::new(self.ptr, dim, broadcast_strides)) }
23072304
}
23082305

src/iterators/mod.rs

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,13 +139,9 @@ impl<A> DoubleEndedIterator for Baseiter<A, Ix1>
139139
#[inline]
140140
fn next_back(&mut self) -> Option<Self::Item>
141141
{
142-
let index = match self.index {
143-
None => return None,
144-
Some(ix) => ix,
145-
};
146142
self.dim[0] -= 1;
147143
let offset = Ix1::stride_offset(&self.dim, &self.strides);
148-
if index == self.dim {
144+
if self.index? == self.dim {
149145
self.index = None;
150146
}
151147

0 commit comments

Comments
 (0)