perf: use .next_back() to get the last component

Using `.last()` needs to go through all components first instead of
splitting only the last one.
This commit is contained in:
Samuel Tardieu 2025-01-02 22:36:21 +01:00
parent 9048e80fed
commit 39bdd5eb3f

View file

@ -212,7 +212,7 @@ pub fn run_mergetool_external(
}; };
let temp_dir = new_utf8_temp_dir("jj-resolve-").map_err(ExternalToolError::SetUpDir)?; let temp_dir = new_utf8_temp_dir("jj-resolve-").map_err(ExternalToolError::SetUpDir)?;
let suffix = if let Some(filename) = repo_path.components().last() { let suffix = if let Some(filename) = repo_path.components().next_back() {
let name = filename let name = filename
.to_fs_name() .to_fs_name()
.map_err(|err| err.with_path(repo_path))?; .map_err(|err| err.with_path(repo_path))?;