Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Sergio Gasquez Arcos <[email protected]>
  • Loading branch information
jasta and SergioGasquez committed Feb 1, 2023
1 parent 78ee947 commit 8e45be7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/toolchain/llvm.rs
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ pub struct Llvm {
pub repository_url: String,
/// LLVM Version ["15"].
pub version: String,
/// True if only if the libraries were installed
/// If `true`, only libraries are installed.
minified: bool,
}

Expand All @@ -54,7 +54,7 @@ impl Llvm {
let llvm_path = format!("{}/esp-clang/lib", self.path.to_str().unwrap());
llvm_path
}

/// Gets the binary path of clang
fn get_bin_path(&self) -> String {
#[cfg(windows)]
let llvm_path = format!("{}/esp-clang/bin/clang.exe", self.path.to_str().unwrap());
Expand Down

0 comments on commit 8e45be7

Please sign in to comment.