Skip to content

Commit cfdb976

Browse files
committed
Fix new (non-pedantic) clippy warnings
1 parent c4f8e9b commit cfdb976

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/external_trait_impls/rayon/map.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -679,7 +679,7 @@ mod test_par_map {
679679
fn test_values_mut() {
680680
let vec = vec![(1, 1), (2, 2), (3, 3)];
681681
let mut map: HashMap<_, _> = vec.into_par_iter().collect();
682-
map.par_values_mut().for_each(|value| *value = (*value) * 2);
682+
map.par_values_mut().for_each(|value| *value *= 2);
683683
let values: Vec<_> = map.par_values().cloned().collect();
684684
assert_eq!(values.len(), 3);
685685
assert!(values.contains(&2));

src/external_trait_impls/rayon/raw.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ impl<T: Send> UnindexedProducer for ParDrainProducer<T> {
175175
{
176176
// Make sure to modify the iterator in-place so that any remaining
177177
// elements are processed in our Drop impl.
178-
while let Some(item) = self.iter.next() {
178+
for item in &mut self.iter {
179179
folder = folder.consume(unsafe { item.read() });
180180
if folder.full() {
181181
return folder;
@@ -193,7 +193,7 @@ impl<T> Drop for ParDrainProducer<T> {
193193
fn drop(&mut self) {
194194
// Drop all remaining elements
195195
if mem::needs_drop::<T>() {
196-
while let Some(item) = self.iter.next() {
196+
for item in &mut self.iter {
197197
unsafe {
198198
item.drop();
199199
}

0 commit comments

Comments
 (0)