completion: Split declaration and assignment of variables

Having them on the same line masks failure of the subcommand generating
the value being assigned. Spotted by shellcheck.

Signed-off-by: Philip Withnall <pwithnall@gnome.org>
This commit is contained in:
Philip Withnall 2024-04-16 14:05:04 +01:00
parent a7ad2a4c3a
commit d3d811f4e5
No known key found for this signature in database
GPG Key ID: DCDF5885B1F3ED73
2 changed files with 6 additions and 3 deletions

View File

@ -9,9 +9,11 @@
__gdbus() {
local IFS=$'\n'
local cur="$(_get_cword :)"
local cur
cur="$(_get_cword :)"
local suggestions="$(gdbus complete "${COMP_LINE}" "${COMP_POINT}")"
local suggestions
suggestions="$(gdbus complete "${COMP_LINE}" "${COMP_POINT}")"
COMPREPLY=($(compgen -W "$suggestions" -- "$cur"))
# Remove colon-word prefix from COMPREPLY items

View File

@ -71,7 +71,8 @@ __gio_location() {
local name size type
while IFS=$'\t' read name size type; do
# Escape name properly
local escaped_name="$(printf "%q" "$name")"
local escaped_name
escaped_name="$(printf "%q" "$name")"
# Append slash for directories and space for files
if [[ "$type" == "(directory)" ]]; then