Fix most clippy lints

Author: newAM

cortex-m-rt/macros/src/lib.rs

@@ -28,10 +28,7 @@ pub fn entry(args: TokenStream, input: TokenStream) -> TokenStream {
         && f.sig.variadic.is_none()
         && match f.sig.output {
             ReturnType::Default => false,
-            ReturnType::Type(_, ref ty) => match **ty {
-                Type::Never(_) => true,
-                _ => false,
-            },
+            ReturnType::Type(_, ref ty) => matches!(**ty, Type::Never(_)),
         };
 
     if !valid_signature {
@@ -159,7 +156,7 @@ pub fn exception(args: TokenStream, input: TokenStream) -> TokenStream {
             Exception::DefaultHandler | Exception::HardFault | Exception::NonMaskableInt => {
                 // These are unsafe to define.
                 let name = if exn == Exception::DefaultHandler {
-                    format!("`DefaultHandler`")
+                    "`DefaultHandler`".to_string()
                 } else {
                     format!("`{:?}` handler", exn)
                 };
@@ -252,10 +249,7 @@ pub fn exception(args: TokenStream, input: TokenStream) -> TokenStream {
                 && f.sig.variadic.is_none()
                 && match f.sig.output {
                     ReturnType::Default => false,
-                    ReturnType::Type(_, ref ty) => match **ty {
-                        Type::Never(_) => true,
-                        _ => false,
-                    },
+                    ReturnType::Type(_, ref ty) => matches!(**ty, Type::Never(_)),
                 };
 
             if !valid_signature {
@@ -557,7 +551,7 @@ fn extract_static_muts(
     let mut seen = HashSet::new();
     let mut statics = vec![];
     let mut stmts = vec![];
-    while let Some(stmt) = istmts.next() {
+    for stmt in istmts.by_ref() {
         match stmt {
             Stmt::Item(Item::Static(var)) => {
                 if var.mutability.is_some() {
@@ -622,7 +616,7 @@ fn check_attr_whitelist(attrs: &[Attribute], caller: WhiteListCaller) -> Result<
 
     'o: for attr in attrs {
         for val in whitelist {
-            if eq(&attr, &val) {
+            if eq(attr, val) {
                 continue 'o;
             }
         }

xtask/src/lib.rs

@@ -232,15 +232,15 @@ pub fn check_host_side() {
     {
         let a = VectActive::from(19).unwrap();
         let b = VectActive::from(20).unwrap();
-        assert_eq!(a < b, true);
+        assert!(a < b);
     }
 
     // check TryFrom
     {
         use core::convert::TryInto;
         use std::convert::TryFrom;
 
-        let lts: LocalTimestampOptions = (16 as u8).try_into().unwrap();
+        let lts: LocalTimestampOptions = (16_u8).try_into().unwrap();
         assert_eq!(lts, LocalTimestampOptions::EnabledDiv16);
 
         assert!(LocalTimestampOptions::try_from(42).is_err());

xtask/src/main.rs

@@ -2,8 +2,8 @@ use std::{env, process};
 use xtask::{assemble_blobs, check_blobs, check_host_side};
 
 fn main() {
-    let subcommand = env::args().skip(1).next();
-    match subcommand.as_ref().map(|s| &**s) {
+    let subcommand = env::args().nth(1);
+    match subcommand.as_deref() {
         Some("assemble") => assemble_blobs(),
         Some("check-blobs") => check_blobs(),
         Some("check-host-side") => check_host_side(),