diff --git a/helix-stdx/src/path.rs b/helix-stdx/src/path.rs index 42d7a9b60..c514f9f27 100644 --- a/helix-stdx/src/path.rs +++ b/helix-stdx/src/path.rs @@ -10,17 +10,21 @@ /// Replaces users home directory from `path` with tilde `~` if the directory /// is available, otherwise returns the path unchanged. -pub fn fold_home_dir(path: &Path) -> PathBuf { +pub fn fold_home_dir<'a, P>(path: P) -> Cow<'a, Path> +where + P: Into>, +{ + let path = path.into(); if let Ok(home) = home_dir() { if let Ok(stripped) = path.strip_prefix(&home) { let mut path = OsString::with_capacity(2 + stripped.as_os_str().len()); path.push("~/"); path.push(stripped); - return PathBuf::from(path); + return Cow::Owned(PathBuf::from(path)); } } - path.to_path_buf() + path } /// Expands tilde `~` into users home directory if available, otherwise returns the path @@ -129,18 +133,21 @@ pub fn canonicalize(path: impl AsRef) -> PathBuf { normalize(path) } -pub fn get_relative_path(path: impl AsRef) -> PathBuf { - let path = PathBuf::from(path.as_ref()); - let path = if path.is_absolute() { +pub fn get_relative_path<'a, P>(path: P) -> Cow<'a, Path> +where + P: Into>, +{ + let path = path.into(); + if path.is_absolute() { let cwdir = normalize(current_working_dir()); - normalize(&path) - .strip_prefix(cwdir) - .map(PathBuf::from) - .unwrap_or(path) - } else { - path - }; - fold_home_dir(&path) + if let Ok(stripped) = normalize(&path).strip_prefix(cwdir) { + return Cow::Owned(PathBuf::from(stripped)); + } + + return fold_home_dir(path); + } + + path } /// Returns a truncated filepath where the basepart of the path is reduced to the first diff --git a/helix-term/src/ui/statusline.rs b/helix-term/src/ui/statusline.rs index c3464067f..7437cbd07 100644 --- a/helix-term/src/ui/statusline.rs +++ b/helix-term/src/ui/statusline.rs @@ -482,7 +482,7 @@ fn render_file_base_name(context: &mut RenderContext, write: F) let rel_path = context.doc.relative_path(); let path = rel_path .as_ref() - .and_then(|p| p.as_path().file_name().map(|s| s.to_string_lossy())) + .and_then(|p| p.file_name().map(|s| s.to_string_lossy())) .unwrap_or_else(|| SCRATCH_BUFFER_NAME.into()); format!(" {} ", path) }; diff --git a/helix-view/src/document.rs b/helix-view/src/document.rs index 4e7b1de9f..5f3595eeb 100644 --- a/helix-view/src/document.rs +++ b/helix-view/src/document.rs @@ -1685,7 +1685,7 @@ pub fn selections(&self) -> &HashMap { &self.selections } - pub fn relative_path(&self) -> Option { + pub fn relative_path(&self) -> Option> { self.path .as_deref() .map(helix_stdx::path::get_relative_path)