Merge pull request #8 from n-thumann/make_clippy_happy

n-thumann · web-flow · commit 075185114103 · 2025-11-30T21:35:29.000+01:00
Change: Remove unneeded return statements
diff --git a/src/config.rs b/src/config.rs
@@ -18,11 +18,10 @@ pub struct Config {
 }
 
 fn join_with_comma<T: ToString>(ips: &[T]) -> String {
-    return ips
-        .iter()
+    ips.iter()
         .map(|ip| ip.to_string())
         .collect::<Vec<String>>()
-        .join(", ");
+        .join(", ")
 }
 
 impl fmt::Display for Config {
diff --git a/src/nordvpn/server.rs b/src/nordvpn/server.rs
@@ -11,7 +11,7 @@ pub struct Server {
 
 impl Server {
     pub fn supports_ipv6(&self) -> bool {
-        return self.ip_addresses.iter().any(IpAddr::is_ipv6);
+        self.ip_addresses.iter().any(IpAddr::is_ipv6)
     }
 
     pub fn endpoint(&self) -> String {

Original file line number	Diff line number	Diff line change
`@@ -18,11 +18,10 @@ pub struct Config {`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`fn join_with_comma<T: ToString>(ips: &[T]) -> String {`
`21`		`- return ips`
`22`		`- .iter()`
	`21`	`+ ips.iter()`
`23`	`22`	`.map(\|ip\| ip.to_string())`
`24`	`23`	`.collect::<Vec<String>>()`
`25`		`- .join(", ");`
	`24`	`+ .join(", ")`
`26`	`25`	`}`
`27`	`26`
`28`	`27`	`impl fmt::Display for Config {`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ pub struct Server {`
`11`	`11`
`12`	`12`	`impl Server {`
`13`	`13`	`pub fn supports_ipv6(&self) -> bool {`
`14`		`- return self.ip_addresses.iter().any(IpAddr::is_ipv6);`
	`14`	`+ self.ip_addresses.iter().any(IpAddr::is_ipv6)`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`pub fn endpoint(&self) -> String {`