From 1f65e9029b8c9d0fbe6c60d8fee249c57fd3f3fa Mon Sep 17 00:00:00 2001 From: Evan Benn Date: Fri, 9 Sep 2022 15:35:28 +1000 Subject: flashrom_tester: Fix cargo check and clippy warnings Change-Id: I50c5af61e06df1bb6956f347cb6806a7eca6ce0e Signed-off-by: Evan Benn Reviewed-on: https://review.coreboot.org/c/flashrom/+/67472 Tested-by: build bot (Jenkins) Reviewed-by: Peter Marheine Reviewed-by: Angel Pons --- util/flashrom_tester/src/tests.rs | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) (limited to 'util/flashrom_tester/src/tests.rs') diff --git a/util/flashrom_tester/src/tests.rs b/util/flashrom_tester/src/tests.rs index 7d746b5d..1527a6e6 100644 --- a/util/flashrom_tester/src/tests.rs +++ b/util/flashrom_tester/src/tests.rs @@ -43,8 +43,8 @@ use std::fs::{self, File}; use std::io::{BufRead, Write}; use std::sync::atomic::AtomicBool; -const LAYOUT_FILE: &'static str = "/tmp/layout.file"; -const ELOG_FILE: &'static str = "/tmp/elog.file"; +const LAYOUT_FILE: &str = "/tmp/layout.file"; +const ELOG_FILE: &str = "/tmp/elog.file"; /// Iterate over tests, yielding only those tests with names matching filter_names. /// @@ -79,6 +79,7 @@ fn filter_tests<'n, 't: 'n, T: TestCase>( /// test_names is the case-insensitive names of tests to run; if None, then all /// tests are run. Provided names that don't match any known test will be logged /// as a warning. +#[allow(clippy::or_fun_call)] // This is used for to_string here and we don't care. pub fn generic<'a, TN: Iterator>( cmd: &dyn Flashrom, fc: FlashChip, @@ -132,11 +133,8 @@ pub fn generic<'a, TN: Iterator>( // Limit the tests to only those requested, unless none are requested // in which case all tests are included. - let mut filter_names: Option> = if let Some(names) = test_names { - Some(names.map(|s| s.to_lowercase()).collect()) - } else { - None - }; + let mut filter_names: Option> = + test_names.map(|names| names.map(|s| s.to_lowercase()).collect()); let tests = filter_tests(tests, &mut filter_names); let chip_name = cmd @@ -153,15 +151,15 @@ pub fn generic<'a, TN: Iterator>( warn!("No test matches filter name \"{}\"", leftover); } - let os_rel = sys_info::os_release().unwrap_or("".to_string()); + let os_release = sys_info::os_release().unwrap_or("".to_string()); let system_info = cros_sysinfo::system_info().unwrap_or("".to_string()); let bios_info = cros_sysinfo::bios_info().unwrap_or("".to_string()); let meta_data = tester::ReportMetaData { - chip_name: chip_name, - os_release: os_rel, - system_info: system_info, - bios_info: bios_info, + chip_name, + os_release, + system_info, + bios_info, }; tester::collate_all_test_runs(&results, meta_data, output_format); Ok(()) -- cgit v1.2.3