Merge pull request #239 from taskchampion/fix-new-clippy

fix new clippy warnings
This commit is contained in:
Dustin J. Mitchell
2021-05-17 21:34:03 -04:00
committed by GitHub
2 changed files with 3 additions and 11 deletions

View File

@@ -144,10 +144,8 @@ impl TaskDb {
// if renumbering, clear the working set and re-add
if renumber {
txn.clear_working_set()?;
for elt in new_ws.drain(0..new_ws.len()) {
if let Some(uuid) = elt {
txn.add_to_working_set(uuid)?;
}
for uuid in new_ws.drain(0..new_ws.len()).flatten() {
txn.add_to_working_set(uuid)?;
}
} else {
// ..otherwise, just clear the None items determined above from the working set

View File

@@ -58,13 +58,7 @@ impl WorkingSet {
self.by_index
.iter()
.enumerate()
.filter_map(|(index, uuid)| {
if let Some(uuid) = uuid {
Some((index, *uuid))
} else {
None
}
})
.filter_map(|(index, uuid)| uuid.as_ref().map(|uuid| (index, *uuid)))
}
}