h/fish-shell
1
Fork 0
mirror of https://github.com/fish-shell/fish-shell.git synced 2026-06-08 02:31:18 -03:00
Code Issues Packages Projects Releases Wiki Activity

Tweak replacing completions for directory-variable name

Browse Source 
If we're only inserting a prefix of the variable name we're not gonna
add a space. We don't want to add a slash either.
This commit is contained in:
Johannes Altmanninger
2025-02-11 22:17:09 +01:00
parent 7197bc7760
commit b4b0b42792
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/reader.rs
View File

@@ -6057,20 +6057,21 @@ pub fn completion_apply_to_command_line(
escape_flags.insert(EscapeFlags::NO_TILDE); escape_flags.insert(EscapeFlags::NO_TILDE);
} }
let maybe_slash = |trailer: Option<char>, token: &wstr| -> Option<char> { let maybe_add_slash = |trailer: &mut char, token: &wstr| {
let mut expanded = token.to_owned(); let mut expanded = token.to_owned();
if expand_one(&mut expanded, ExpandFlags::FAIL_ON_CMDSUBST, ctx, None) if expand_one(&mut expanded, ExpandFlags::FAIL_ON_CMDSUBST, ctx, None)
&& wstat(&expanded).is_ok_and(|md| md.is_dir()) && wstat(&expanded).is_ok_and(|md| md.is_dir())
{ {
return Some('/'); *trailer = '/';
} }
trailer
}; };
if do_replace_token { if do_replace_token {
if is_variable_name { if is_variable_name {
assert!(!do_escape); assert!(!do_escape);
trailer = maybe_slash(trailer, val_str); if let Some(trailer) = trailer.as_mut() {
maybe_add_slash(trailer, val_str);
}
} }
let mut move_cursor = 0; let mut move_cursor = 0;
let (range, _) = parse_util_token_extent(command_line, cursor_pos); let (range, _) = parse_util_token_extent(command_line, cursor_pos);
@@ -6153,7 +6154,7 @@ pub fn completion_apply_to_command_line(
if let Some(mut trailer) = trailer { if let Some(mut trailer) = trailer {
if is_variable_name { if is_variable_name {
let (tok, _) = parse_util_token_extent(command_line, cursor_pos); let (tok, _) = parse_util_token_extent(command_line, cursor_pos);
trailer = maybe_slash(Some(trailer), &result[tok.start..new_cursor_pos]).unwrap(); maybe_add_slash(&mut trailer, &result[tok.start..new_cursor_pos]);
} }
if trailer != '/' if trailer != '/'
&& quote.is_some() && quote.is_some()