use tokio async to capture clang tools' version

Author: 2bndy5

cpp-linter/src/clang_tools/mod.rs

@@ -4,9 +4,7 @@
 use std::{
     env::current_dir,
     fmt::{self, Display},
-    fs,
     path::{Path, PathBuf},
-    process::Command,
     sync::{Arc, Mutex},
 };
 
@@ -15,7 +13,7 @@ use anyhow::{anyhow, Context, Result};
 use git2::{DiffOptions, Patch};
 use regex::Regex;
 use semver::Version;
-use tokio::task::JoinSet;
+use tokio::{fs, process::Command, task::JoinSet};
 use which::{which, which_in};
 
 // project-specific modules/crates
@@ -115,8 +113,8 @@ impl ClangTool {
     }
 
     /// Run `clang-tool --version`, then extract and return the version number.
-    fn capture_version(clang_tool: &PathBuf) -> Result<String> {
-        let output = Command::new(clang_tool).arg("--version").output()?;
+    async fn capture_version(clang_tool: &PathBuf) -> Result<String> {
+        let output = Command::new(clang_tool).arg("--version").output().await?;
         let stdout = String::from_utf8_lossy(&output.stdout);
         let version_pattern = Regex::new(r"(?i)version[^\d]*([\d.]+)").unwrap();
         let captures = version_pattern.captures(&stdout).ok_or(anyhow!(
@@ -212,7 +210,7 @@ pub async fn capture_clang_tools_output(
     // info as debugging output.
     if clang_params.tidy_checks != "-*" {
         let exe_path = ClangTool::ClangTidy.get_exe_path(version)?;
-        let version_found = ClangTool::capture_version(&exe_path)?;
+        let version_found = ClangTool::capture_version(&exe_path).await?;
         log::debug!(
             "{} --version: v{version_found}",
             &exe_path.to_string_lossy()
@@ -222,7 +220,7 @@ pub async fn capture_clang_tools_output(
     }
     if !clang_params.style.is_empty() {
         let exe_path = ClangTool::ClangFormat.get_exe_path(version)?;
-        let version_found = ClangTool::capture_version(&exe_path)?;
+        let version_found = ClangTool::capture_version(&exe_path).await?;
         log::debug!(
             "{} --version: v{version_found}",
             &exe_path.to_string_lossy()
@@ -233,7 +231,7 @@ pub async fn capture_clang_tools_output(
 
     // parse database (if provided) to match filenames when parsing clang-tidy's stdout
     if let Some(db_path) = &clang_params.database {
-        if let Ok(db_str) = fs::read(db_path.join("compile_commands.json")) {
+        if let Ok(db_str) = fs::read(db_path.join("compile_commands.json")).await {
             clang_params.database_json = Some(
                 // A compilation database should be UTF-8 encoded, but file paths are not; use lossy conversion.
                 serde_json::from_str::<Vec<CompilationUnit>>(&String::from_utf8_lossy(&db_str))