Fix clippy lint in agb tools (#581)

Clippy lint was getting upset that we were pointlessly cloning. So stop
doing that (even though the lint seems to have a bug, but I'd like the
build to start working again).

- [x]  no changelog update needed
This commit is contained in:
Gwilym Inzani 2024-03-09 12:18:38 +00:00 committed by GitHub
commit d804edb1ec
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -1,19 +1,21 @@
use std::{env, path::PathBuf}; use std::{
env,
path::{Path, PathBuf},
};
#[derive(Debug)] #[derive(Debug)]
pub struct FindRootDirectoryError; pub struct FindRootDirectoryError;
pub fn find_agb_root_directory() -> Result<PathBuf, FindRootDirectoryError> { pub fn find_agb_root_directory() -> Result<PathBuf, FindRootDirectoryError> {
let mut current_path = env::current_dir().map_err(|_| FindRootDirectoryError)?; let current_path = env::current_dir().map_err(|_| FindRootDirectoryError)?;
while !current_path.clone().join("justfile").exists() { let mut search_path: &Path = &current_path;
current_path = current_path
.parent() while !search_path.join("justfile").exists() {
.ok_or(FindRootDirectoryError)? search_path = search_path.parent().ok_or(FindRootDirectoryError)?;
.to_owned();
} }
Ok(current_path) Ok(search_path.to_owned())
} }
#[cfg(test)] #[cfg(test)]
@ -22,6 +24,7 @@ mod tests {
#[test] #[test]
fn find_agb_root_directory_works() { fn find_agb_root_directory_works() {
find_agb_root_directory().unwrap(); let agb_root = find_agb_root_directory().unwrap();
assert!(agb_root.join("justfile").exists());
} }
} }