Skip to content

Commit

Permalink
fix clippy warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
laurci committed Mar 20, 2024
1 parent 058a62b commit 1c2efbb
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions framework/meta/src/cmd/standalone/test_coverage/llvm_cov.rs
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,8 @@ pub fn combine_instrumentation_results(
pub fn export_coverage_summary(
root_dir: &str,
profdata_file: &str,
binary_files: &Vec<String>,
ignore_filename_regex: &Vec<String>,
binary_files: &[String],
ignore_filename_regex: &[String],
) -> Result<Coverage, TestCoverageError> {
let objects = binary_files
.iter()
Expand All @@ -103,7 +103,7 @@ pub fn export_coverage_summary(
.arg("export")
.args(&objects)
.args(&ignore_filename_regex)
.args(&[
.args([
&format!("--instr-profile={}", profdata_file),
"--summary-only",
"--format=text",
Expand Down
4 changes: 2 additions & 2 deletions framework/meta/src/cmd/standalone/test_coverage/run.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ pub fn run_test_coverage(
root_path: &str,
output_path: &str,
output_format: &TestCoverageOutputFormat,
ignore_filename_regex: &Vec<String>,
ignore_filename_regex: &[String],
) -> Result<(), TestCoverageError> {
ensure_dependencies_in_path()?;

Expand All @@ -44,7 +44,7 @@ pub fn run_test_coverage(

match output_format {
TestCoverageOutputFormat::Markdown => {
render_coverage(&mut output, &coverage, &root_path);
render_coverage(&mut output, &coverage, root_path);
},
TestCoverageOutputFormat::Json => {
output = serde_json::to_string_pretty(&coverage).unwrap();
Expand Down

0 comments on commit 1c2efbb

Please sign in to comment.