Fix some minor clippy lints

Author: ilmanzo

src/cli.rs

@@ -87,7 +87,7 @@ fn parse_signal(arg: &str) -> Result<Signal, std::io::Error> {
         Ok(str_arg) => {
             let signal_str = str_arg.to_uppercase();
             let signal = Signal::from_str(signal_str.as_str())?;
-            return Ok(signal);
+            Ok(signal)
         }
         Err(e) => Err(std::io::Error::new(std::io::ErrorKind::Other, e)),
     }

src/docker.rs

@@ -17,7 +17,7 @@ impl DockerContainer {
     ///
     /// * `name` - A container name.
     /// * `signal` - A enum value representing the signal type.
-    pub fn kill_container(name: &String, signal: Signal) -> Result<(), Error> {
+    pub fn kill_container(name: &str, signal: Signal) -> Result<(), Error> {
         let rt = Runtime::new()?;
         rt.block_on(async {
             let docker = Docker::connect_with_socket_defaults()
@@ -63,8 +63,8 @@ impl DockerContainer {
                         .as_ref()?
                         .first()
                         .map(|name| DockerContainer {
-                            name: if name.starts_with('/') {
-                                name[1..].to_string()
+                            name: if let Some(stripped) = name.strip_prefix('/') {
+                                stripped.to_string()
                             } else {
                                 name.clone()
                             },

src/killport.rs

@@ -69,10 +69,7 @@ impl Killable for DockerContainer {
     ///
     /// * `signal` - A enum value representing the signal type.
     fn kill(&self, signal: Signal) -> Result<bool, Error> {
-        if let Err(err) = Self::kill_container(&self.name, signal) {
-            return Err(err);
-        }
-
+        Self::kill_container(&self.name, signal)?;
         Ok(true)
     }
 

src/linux.rs

@@ -98,7 +98,7 @@ pub fn find_target_processes(port: u16) -> Result<Vec<NativeProcess>, Error> {
                             debug!("Found process '{}' with PID {}", name, process.pid());
                             target_pids.push(NativeProcess {
                                 pid: Pid::from_raw(process.pid),
-                                name: name,
+                                name,
                             });
                         }
                     }