clippy: derive Default

djmitche · Dec 25, 2023 · 8286105 · 8286105
1 parent 2eccd07
commit 8286105
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/taskchampion/lib/src/string.rs b/taskchampion/lib/src/string.rs
@@ -116,21 +116,16 @@ impl TCString {
     }
 }
 
-#[derive(PartialEq, Eq, Debug)]
+#[derive(PartialEq, Eq, Debug, Default)]
 pub enum RustString<'a> {
+    #[default]
     Null,
     CString(CString),
     CStr(&'a CStr),
     String(String),
     Bytes(Vec<u8>),
 }
 
-impl<'a> Default for RustString<'a> {
-    fn default() -> Self {
-        RustString::Null
-    }
-}
-
 impl PassByValue for TCString {
     type RustType = RustString<'static>;
 

diff --git a/taskchampion/taskchampion/src/server/cloud/server.rs b/taskchampion/taskchampion/src/server/cloud/server.rs
@@ -157,8 +157,7 @@ impl<SVC: Service> CloudServer<SVC> {
     /// Get the possible child versions of the given parent version, based only on the object
     /// names.
     fn get_child_versions(&mut self, parent_version_id: &VersionId) -> Result<Vec<VersionId>> {
-        self
-            .service
+        self.service
             .list(format!("v-{}-", parent_version_id.as_simple()).as_bytes())
             .filter_map(|res| match res {
                 Ok(ObjectInfo { name, .. }) => {
@@ -280,9 +279,7 @@ impl<SVC: Service> CloudServer<SVC> {
             .service
             .list(b"s-")
             .filter_map(|res| match res {
-                Ok(ObjectInfo { name, .. }) => {
-                    Self::parse_snapshot_name(&name).map(Ok)
-                }
+                Ok(ObjectInfo { name, .. }) => Self::parse_snapshot_name(&name).map(Ok),
                 Err(e) => Some(Err(e)),
             })
             .collect::<Result<HashSet<_>>>()?;