Skip to content

Commit 54364dc

Browse files
committed
[rust] Minor clippy fixes
1 parent 7497552 commit 54364dc

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

rust/src/files.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ pub fn uncompress_sfx(compressed_file: &str, target: &Path, log: &Logger) -> Res
189189

190190
let zip_parent_str = path_to_string(zip_parent);
191191
let core_str = format!(r"{}\core", zip_parent_str);
192-
move_folder_content(&core_str, &target, &log)?;
192+
move_folder_content(&core_str, target, log)?;
193193

194194
Ok(())
195195
}
@@ -304,7 +304,7 @@ pub fn uncompress_deb(
304304
fs::remove_file(Path::new(&link)).unwrap_or_default();
305305
}
306306

307-
move_folder_content(&opt_edge_str, &target, &log)?;
307+
move_folder_content(&opt_edge_str, target, log)?;
308308

309309
Ok(())
310310
}

rust/src/lib.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ pub trait SeleniumManager {
192192
let driver_name_with_extension = self.get_driver_name_with_extension();
193193

194194
let mut lock = Lock::acquire(
195-
&self.get_logger(),
195+
self.get_logger(),
196196
&driver_path_in_cache,
197197
Some(driver_name_with_extension.clone()),
198198
)?;
@@ -325,7 +325,7 @@ pub trait SeleniumManager {
325325
}
326326

327327
let browser_path_in_cache = self.get_browser_path_in_cache()?;
328-
let mut lock = Lock::acquire(&self.get_logger(), &browser_path_in_cache, None)?;
328+
let mut lock = Lock::acquire(self.get_logger(), &browser_path_in_cache, None)?;
329329
if !lock.exists() && browser_binary_path.exists() {
330330
self.get_logger().debug(format!(
331331
"Browser already in cache: {}",
@@ -350,7 +350,7 @@ pub trait SeleniumManager {
350350
uncompress(
351351
&driver_zip_file,
352352
&browser_path_in_cache,
353-
&self.get_logger(),
353+
self.get_logger(),
354354
self.get_os(),
355355
None,
356356
browser_label_for_download,

rust/src/lock.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ use crate::files::{create_parent_path_if_not_exists, create_path_if_not_exists};
2525
use fs2::FileExt;
2626
use std::fs;
2727

28-
thread_local!(static LOCK_PATH: RefCell<Option<PathBuf>> = RefCell::new(None));
28+
thread_local!(static LOCK_PATH: RefCell<Option<PathBuf>> = const { RefCell::new(None) });
2929

3030
const LOCK_FILE: &str = "sm.lock";
3131

rust/tests/common.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ use std::path::{Path, PathBuf};
2828

2929
#[allow(dead_code)]
3030
pub fn get_selenium_manager() -> Command {
31-
let mut path = PathBuf::from(env!("CARGO_BIN_EXE_selenium-manager"));
31+
let path = PathBuf::from(env!("CARGO_BIN_EXE_selenium-manager"));
3232

3333
if path.exists() {
3434
return Command::new(path);

0 commit comments

Comments
 (0)