mroot/yazi
1
0
Fork 0
mirror of https://github.com/sxyazi/yazi.git synced 2026-07-25 08:41:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix: excessive separator at the end of the path

Browse source
This commit is contained in:
alan910127 2024-11-02 03:46:14 +08:00
parent 13f7d888fd
commit 2840d5ba2b
1 changed files with 16 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
yazi-core/src/tab/commands/copy.rs
View file

@ -97,25 +97,34 @@ fn path_to_os_str(path: &Path, separator: PathSeparator) -> Cow<'_, OsStr> {
use std::path::Component;
let mut s = OsString::with_capacity(path.as_os_str().len());
let mut s = String::with_capacity(path.as_os_str().len());
for component in path.components() {
match component {
Component::RootDir => {}
Component::CurDir => s.push("."),
Component::ParentDir => s.push(".."),
Component::Normal(path) => s.push(path),
Component::CurDir => s.push('.'),
Component::ParentDir => s.push_str(".."),
Component::Normal(path) => s.push_str(&path.to_string_lossy()),
Component::Prefix(prefix) => {
// "C:\foo" => [Prefix("C:"), RootDir, Normal(foo)]
s.push(prefix.as_os_str());
s.push_str(&prefix.as_os_str().to_string_lossy());
// If we push a "/" below, we will met a RootDir and push a "/"
// again resulting in "C://". So we need to skip that.
continue;
}
};
s.push("/");
s.push('/');
}
return Cow::Owned(s);
use std::os::windows::ffi::OsStrExt as _;
use std::path::MAIN_SEPARATOR;
let path_ends_with_main_sep =
path.as_os_str().encode_wide().last() == Some(MAIN_SEPARATOR as u16);
if !path_ends_with_main_sep && s != "/" && s.ends_with('/') {
s.pop();
}
return Cow::Owned(OsString::from(s));
}
#[cfg(test)]