Edition-2018-fied code and fixed a lot of clippy lints

Signed-off-by: Daniel Egger <daniel@eggers-club.de>

Author: therealprof

ci/svd2rust-regress/Cargo.toml

@@ -1,11 +1,12 @@
 [package]
+edition = "2018"
 name = "svd2rust-regress"
 version = "0.1.0"
 authors = ["James Munns <james.munns@gmail.com>", "The svd2rust developers"]
 
 [dependencies]
-reqwest = "0.8"
-rayon = "1.0"
+reqwest = "0.9"
+rayon = "1.1"
 structopt = "0.2"
-error-chain = "0.11"
-inflections = "1.1.0"
+error-chain = "0.12"
+inflections = "1.1"

ci/svd2rust-regress/src/main.rs

@@ -1,9 +1,5 @@
 #[macro_use]
 extern crate error_chain;
-extern crate inflections;
-extern crate rayon;
-extern crate reqwest;
-extern crate structopt;
 
 mod errors;
 mod svd_test;
@@ -176,8 +172,7 @@ fn main() {
             // FIXME: Use Option::filter instead when stable, rust-lang/rust#45860
             if default_rustfmt
                 .iter()
-                .filter(|p| p.is_file())
-                .next()
+                .find(|p| p.is_file())
                 .is_none()
             {
                 panic!("No rustfmt found");
@@ -264,8 +259,8 @@ fn main() {
             Err(e) => {
                 any_fails.store(true, Ordering::Release);
                 let additional_info = if opt.verbose > 0 {
-                    match e.kind() {
-                        &errors::ErrorKind::ProcessFailed(_, _, Some(ref stderr), ref previous_processes_stderr) => {
+                    match *e.kind() {
+                        errors::ErrorKind::ProcessFailed(_, _, Some(ref stderr), ref previous_processes_stderr) => {
                             let mut buf = String::new(); 
                             if opt.verbose > 1 {
                                 for stderr in previous_processes_stderr {

ci/svd2rust-regress/src/svd_test.rs

@@ -1,10 +1,10 @@
-use errors::*;
+use crate::errors::*;
 use reqwest;
 use std::fs::{self, File, OpenOptions};
 use std::io::prelude::*;
 use std::path::PathBuf;
 use std::process::{Command, Output};
-use tests::TestCase;
+use crate::tests::TestCase;
 
 static CRATES_ALL: &[&str] = &["bare-metal = \"0.2.0\"", "vcell = \"0.1.0\""];
 static CRATES_MSP430: &[&str] = &["msp430 = \"0.1.0\""];
@@ -42,7 +42,7 @@ trait CommandHelper {
         name: &str,
         stdout: Option<&PathBuf>,
         stderr: Option<&PathBuf>,
-        previous_processes_stderr: &Vec<PathBuf>,
+        previous_processes_stderr: &[PathBuf],
     ) -> Result<()>;
 }
 
@@ -53,7 +53,7 @@ impl CommandHelper for Output {
         name: &str,
         stdout: Option<&PathBuf>,
         stderr: Option<&PathBuf>,
-        previous_processes_stderr: &Vec<PathBuf>,
+        previous_processes_stderr: &[PathBuf],
     ) -> Result<()> {
         if let Some(out) = stdout {
             let out_payload = String::from_utf8_lossy(&self.stdout);
@@ -70,7 +70,7 @@ impl CommandHelper for Output {
                 ErrorKind::ProcessFailed(name.into(),
                     stdout.cloned(),
                     stderr.cloned(),
-                    previous_processes_stderr.clone(),
+                    previous_processes_stderr.to_vec(),
                 ).into()
             );
         }
@@ -109,7 +109,7 @@ pub fn test(t: &TestCase, bin_path: &PathBuf, rustfmt_bin_path: Option<&PathBuf>
         .arg(&chip_dir)
         .output()
         .chain_err(|| "Failed to cargo init")?
-        .capture_outputs(true, "cargo init", None, None, &vec![])?;
+        .capture_outputs(true, "cargo init", None, None, &[])?;
 
     // Add some crates to the Cargo.toml of our new project
     let svd_toml = path_helper_base(&chip_dir, &["Cargo.toml"]);
@@ -119,7 +119,7 @@ pub fn test(t: &TestCase, bin_path: &PathBuf, rustfmt_bin_path: Option<&PathBuf>
         .open(svd_toml)
         .chain_err(|| "Failed to open Cargo.toml for appending")?;
 
-    use tests::Architecture::*;
+    use crate::tests::Architecture::*;
     let crates = CRATES_ALL
         .iter()
         .chain(match &t.arch {
@@ -154,10 +154,10 @@ pub fn test(t: &TestCase, bin_path: &PathBuf, rustfmt_bin_path: Option<&PathBuf>
     // If the architecture is cortex-m we move the generated lib.rs file to src/
     let lib_rs_file = path_helper_base(&chip_dir, &["src", "lib.rs"]);
     let svd2rust_err_file = path_helper_base(&chip_dir, &["svd2rust.err.log"]);
-    let target = match &t.arch {
-        &CortexM => "cortex-m",
-        &Msp430 => "msp430",
-        &RiscV => "riscv",
+    let target = match t.arch {
+        CortexM => "cortex-m",
+        Msp430 => "msp430",
+        RiscV => "riscv",
     };
     let mut svd2rust_bin = Command::new(bin_path);
     if nightly {
@@ -175,16 +175,13 @@ pub fn test(t: &TestCase, bin_path: &PathBuf, rustfmt_bin_path: Option<&PathBuf>
         "svd2rust",
         if t.arch != CortexM { Some(&lib_rs_file) } else { None }, // use Option.filter
         Some(&svd2rust_err_file),
-        &vec![],
+        &[],
     )?;
     process_stderr_paths.push(svd2rust_err_file);
 
-    match &t.arch {
-        &CortexM => {
-            // TODO: Give error the path to stderr
-            fs::rename(path_helper_base(&chip_dir, &["lib.rs"]), &lib_rs_file).chain_err(|| "While moving lib.rs file")?
-        }
-        _ => (),
+    if let CortexM = t.arch {
+        // TODO: Give error the path to stderr
+        fs::rename(path_helper_base(&chip_dir, &["lib.rs"]), &lib_rs_file).chain_err(|| "While moving lib.rs file")?
     }
 
     let rustfmt_err_file = path_helper_base(&chip_dir, &["rustfmt.err.log"]);

ci/svd2rust-regress/src/tests.rs

@@ -74,7 +74,7 @@ use self::RunWhen::*;
 
 /// List of chars that some vendors use in their peripheral/field names but
 /// that are not valid in Rust ident
-const BLACKLIST_CHARS: &'static [char] = &['(', ')', '[', ']'];
+const BLACKLIST_CHARS: &[char] = &['(', ')', '[', ']'];
 
 /// Lovingly stolen from `svd2rust`. Probably could be `Cow`
 pub trait ToSanitizedSnakeCase {
@@ -96,16 +96,16 @@ impl ToSanitizedSnakeCase for str {
 
         match s.chars().next().unwrap_or('\0') {
             '0' | '1' | '2' | '3' | '4' | '5' | '6' | '7' | '8' | '9' => {
-                String::from(format!("_{}", s.to_snake_case()))
+                format!("_{}", s.to_snake_case())
             }
             _ => {
                 keywords! {
                     s,
                     abstract,
                     alignof,
                     as,
-                    async,
-                    await,
+                    r#async,
+                    r#await,
                     become,
                     box,
                     break,
@@ -145,7 +145,7 @@ impl ToSanitizedSnakeCase for str {
                     super,
                     trait,
                     true,
-                    try,
+                    r#try,
                     type,
                     typeof,
                     unsafe,
@@ -166,7 +166,7 @@ impl ToSanitizedSnakeCase for str {
 }
 
 // NOTE: All chip names must be unique!
-pub const TESTS: &'static [&'static TestCase] = &[
+pub const TESTS: &[&TestCase] = &[
     // BAD-SVD missing resetValue
     &TestCase {
         arch: CortexM,