commit 91de33d9f7ee2643c8143536ff35edb121e3ad24
author David Brown <david.brown@linaro.org> Fri Mar 05 15:43:44 2021 -0700
committer David Brown <davidb@davidb.org> Thu Mar 11 07:25:43 2021 -0700
tree ffc1ca97853cb6200dff2d9eca8531a7adc7639a
parent 5b7ed6a831b4156f43a24a6db0fbb15238182ac4

ptest: Some simple cleanups from clippy

Run the clippy Rust linter, and apply the simple cleanups it suggests.

Signed-off-by: David Brown <david.brown@linaro.org>

ptest/src/main.rs

diff --git a/ptest/src/main.rs b/ptest/src/main.rs
index a543e73..053448b 100644
--- a/ptest/src/main.rs
+++ b/ptest/src/main.rs
@@ -43,7 +43,7 @@
     let ncpus = num_cpus::get();
     let limiter = Arc::new(Semaphore::new(ncpus as isize));
 
-    let matrix = Matrix::from_yaml(&workflow)?;
+    let matrix = Matrix::from_yaml(&workflow);
 
     let mut children = vec![];
     let state = State::new(matrix.envs.len());
@@ -71,7 +71,7 @@
         child.join().unwrap();
     }
 
-    println!("");
+    println!();
 
     Ok(())
 }
@@ -97,7 +97,7 @@
         Arc::new(Mutex::new(State {
             running: HashSet::new(),
             done: HashSet::new(),
-            total: total,
+            total,
         }))
     }
 
@@ -174,7 +174,7 @@
 }
 
 impl Matrix {
-    fn from_yaml(yaml: &[Yaml]) -> Result<Matrix> {
+    fn from_yaml(yaml: &[Yaml]) -> Matrix {
         let mut envs = vec![];
 
         let mut all_tests = HashSet::new();
@@ -192,13 +192,7 @@
                     }
                     Some(e) => e,
                 };
-                let fset = match FeatureSet::decode(elt) {
-                    Ok(fset) => fset,
-                    Err(err) => {
-                        warn!("Skipping: {:?}", err);
-                        continue;
-                    }
-                };
+                let fset = FeatureSet::decode(elt);
 
                 if false {
                     // Respect the groupings in the `.workflow.yml` file.
@@ -222,23 +216,23 @@
             }
         }
 
-        Ok(Matrix {
-            envs: envs,
-        })
+        Matrix {
+            envs,
+        }
     }
 }
 
 impl FeatureSet {
-    fn decode(text: &str) -> Result<FeatureSet> {
+    fn decode(text: &str) -> FeatureSet {
         // The github workflow is just a space separated set of values.
         let values: Vec<_> = text
             .split(',')
             .map(|s| s.to_string())
             .collect();
-        Ok(FeatureSet {
+        FeatureSet {
             env: "MULTI_FEATURES".to_string(),
-            values: values,
-        })
+            values,
+        }
     }
 
     /// Run a test for this given feature set.  Output is captured and will be returned if there is