about summary refs log tree commit diff
path: root/Completion
diff options
context:
space:
mode:
authorMarlon Richert <marlon.richert@gmail.com>2023-05-05 14:41:59 +0300
committerOliver Kiddle <opk@zsh.org>2023-05-21 03:24:17 +0200
commitcaa1c38c6f0000b6ed6a6a50e8dca037ad82308a (patch)
tree9411a7ecde6c7ee999a83cea7d73216339d61739 /Completion
parent9a5f213573fbda6a6d23e985113eaeb849d4ee6d (diff)
downloadzsh-caa1c38c6f0000b6ed6a6a50e8dca037ad82308a.tar.gz
zsh-caa1c38c6f0000b6ed6a6a50e8dca037ad82308a.tar.xz
zsh-caa1c38c6f0000b6ed6a6a50e8dca037ad82308a.zip
51348: Fix subscript completion bugs inside ~[...]
When completing inside ~[...] (_with_ the trailing `]` present), the
following bugs occured:
- Subscript completion was skipped entirely when there were one or more
  slashes ('/') in the subscript, which is incorrect, since slashes are
  allowed there.
- Instead of going through _complete, $_comps[-subscript-] was called
  immediately, causing _setup to be skipped.
- If succesful, _main_complete was exited right after, causing
  menu-style, comppostfuncs and other essential completion features to
  be skipped.
Diffstat (limited to 'Completion')
-rw-r--r--Completion/Base/Core/_main_complete22
1 files changed, 9 insertions, 13 deletions
diff --git a/Completion/Base/Core/_main_complete b/Completion/Base/Core/_main_complete
index 169ca1f40..408a66ee3 100644
--- a/Completion/Base/Core/_main_complete
+++ b/Completion/Base/Core/_main_complete
@@ -93,19 +93,15 @@ fi
 if [[ -z "$compstate[quote]" ]]; then
   if [[ -o equals ]] && compset -P 1 '='; then
     compstate[context]=equal
-  elif [[ "$PREFIX" != */* && "$PREFIX[1]" = '~' ]]; then
-    if [[ "$PREFIX" = '~['[^\]]# ]]; then
-      # Inside ~[...] should be treated as a subscript.
-      compset -p 2
-      # To be consistent, we ignore all but the contents of the square
-      # brackets.
-      compset -S '\]*'
-      compstate[context]=subscript
-      [[ -n $_comps[-subscript-] ]] && $_comps[-subscript-] && return
-    else
-      compset -p 1
-      compstate[context]=tilde
-    fi
+  elif [[ "$PREFIX" = \~\[[^]]# ]]; then
+    # Inside ~[...] should be treated as a subscript.
+    compset -p 2
+    # To be consistent, we ignore all but the contents of the square brackets.
+    compset -S '\]*'
+    compstate[context]=subscript
+  elif [[ "$PREFIX" = \~[^/]# ]]; then
+    compset -p 1
+    compstate[context]=tilde
   fi
 fi