Placate Clippy, organize Cargo.toml

Author: joshuamegnauth54

Cargo.toml

@@ -3,14 +3,17 @@ name = "cosmic-screenshot"
 version = "0.1.0"
 edition = "2021"
 
-# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
-
 [dependencies]
-ashpd = { version = "0.6.8", features = ["tracing"] }
+# App
 chrono = "0.4.24"
+clap = { version = "4.4.16", features = ["derive"] }
 dirs = "5.0.1"
 tokio = { version = "1.28.1", features = ["full"] }
-clap = { version = "4.4.16", features = ["derive"] }
-zbus = { version = "3", default-features = false, features = ["tokio"] }
+
+# Logging
 tracing = "0.1"
 tracing-subscriber = { version = "0.3", features = ["env-filter"] }
+
+# Screenshots/D-Bus comm
+ashpd = { version = "0.6.8", features = ["tracing"] }
+zbus = { version = "3", default-features = false, features = ["tokio"] }

src/error.rs

@@ -66,13 +66,9 @@ impl Error {
                     )
                 })
                 .unwrap_or_else(|| "Unable to save screenshot to the Pictures directory".into()),
-            Self::Ashpd(e) => match e {
-                AshpdError::Portal(e) => match e {
-                    PortalError::NotAllowed(msg) => format!("Screenshot not allowed: {msg}"),
-                    _ => "Failed to take screenshot".into(),
-                },
-                _ => "Failed to take screenshot".into(),
-            },
+            Self::Ashpd(AshpdError::Portal(PortalError::NotAllowed(msg))) => {
+                format!("Screenshot not allowed: {msg}")
+            }
             Self::SaveScreenshot { .. } => "Screenshot succeeded but couldn't be saved".into(),
             _ => "Failed to take screenshot".into(),
         }
@@ -86,13 +82,7 @@ impl Error {
 
         match e {
             AshpdError::Response(e) => *e == ResponseError::Cancelled,
-            AshpdError::Portal(e) => {
-                if let PortalError::Cancelled(_) = e {
-                    true
-                } else {
-                    false
-                }
-            }
+            AshpdError::Portal(PortalError::Cancelled(_)) => true,
             _ => false,
         }
     }

src/main.rs

@@ -1,16 +1,13 @@
+#![allow(clippy::too_many_arguments)]
+
 mod error;
 
-use ashpd::desktop::screenshot::{Screenshot, ScreenshotRequest};
+use ashpd::desktop::screenshot::Screenshot;
 use clap::{command, ArgAction, Parser};
 use std::{collections::HashMap, fs, os::unix::fs::MetadataExt, path::PathBuf};
 use tracing::{error, info};
 use tracing_subscriber::{fmt, prelude::*, EnvFilter};
-use zbus::{
-    dbus_proxy,
-    export::futures_util::{future::FutureExt, TryFutureExt},
-    zvariant::Value,
-    Connection,
-};
+use zbus::{dbus_proxy, zvariant::Value, Connection};
 
 use error::Error;
 
@@ -169,7 +166,11 @@ async fn main() -> Result<(), Error> {
             path
         }
         Err(e) => {
-            error!("Screenshot failed with {e}");
+            if !e.cancelled() {
+                error!("Screenshot failed with {e}");
+            } else {
+                info!("Screenshot cancelled");
+            }
             e.to_user_facing()
         }
     };