GothenburgBitFactory/taskwarrior
1
0
Fork 0
mirror of https://github.com/GothenburgBitFactory/taskwarrior.git synced 2025-07-07 20:06:36 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix clippy warnings

Browse source
This commit is contained in:
Dustin J. Mitchell 2020-11-29 14:05:51 -05:00
parent d832b0b859
commit 42d988d601
3 changed files with 4 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

5
sync-server/src/storage/inmemory.rs
View file

@ -39,7 +39,7 @@ impl<'a> StorageTxn for InnerTxn<'a> {
} }
fn new_client(&mut self, client_id: Uuid, latest_version_id: Uuid) -> Fallible<()> { fn new_client(&mut self, client_id: Uuid, latest_version_id: Uuid) -> Fallible<()> {
if let Some(_) = self.0.clients.get(&client_id) { if self.0.clients.get(&client_id).is_some() {
return Err(format_err!("Client {} already exists", client_id)); return Err(format_err!("Client {} already exists", client_id));
} }
self.0 self.0
@ -54,7 +54,8 @@ impl<'a> StorageTxn for InnerTxn<'a> {
latest_version_id: Uuid, latest_version_id: Uuid,
) -> Fallible<()> { ) -> Fallible<()> {
if let Some(client) = self.0.clients.get_mut(&client_id) { if let Some(client) = self.0.clients.get_mut(&client_id) {
Ok(client.latest_version_id = latest_version_id) client.latest_version_id = latest_version_id;
Ok(())
} else { } else {
Err(format_err!("Client {} does not exist", client_id)) Err(format_err!("Client {} does not exist", client_id))
} }

1
taskchampion/src/server/remote.rs
View file

@ -1,7 +1,6 @@
use crate::server::{AddVersionResult, GetVersionResult, HistorySegment, Server, VersionId}; use crate::server::{AddVersionResult, GetVersionResult, HistorySegment, Server, VersionId};
use failure::{format_err, Fallible}; use failure::{format_err, Fallible};
use std::io::Read; use std::io::Read;
use ureq;
use uuid::Uuid; use uuid::Uuid;
pub struct RemoteServer { pub struct RemoteServer {

4
taskchampion/src/task.rs
View file

@ -151,9 +151,7 @@ impl Task {
pub fn is_active(&self) -> bool { pub fn is_active(&self) -> bool {
self.taskmap self.taskmap
.iter() .iter()
.filter(|(k, v)| k.starts_with("start.") && v.is_empty()) .any(|(k, v)| k.starts_with("start.") && v.is_empty())
.next()
.is_some()
} }
pub fn get_modified(&self) -> Option<DateTime<Utc>> { pub fn get_modified(&self) -> Option<DateTime<Utc>> {