*: cleanups

Author: scop

completions/complete

@@ -37,9 +37,9 @@ _complete()
 
     if [[ $cur == -* ]]; then
         # relevant options completion
-        local opts="-a -b -c -d -e -f -g -j -k -o -s -u -v -A -G -W -P -S -X"
-        [[ $1 != compgen ]] && opts+=" -F -C"
-        COMPREPLY=($(compgen -W "$opts" -- "$cur"))
+        local -a opts=(-a -b -c -d -e -f -g -j -k -o -s -u -v -A -G -W -P -S -X)
+        [[ $1 != compgen ]] && opts+=(-F -C)
+        COMPREPLY=($(compgen -W '${opts[@]}' -- "$cur"))
     else
         COMPREPLY=($(compgen -A command -- "$cur"))
     fi

completions/pkg-get

@@ -58,14 +58,15 @@ _pkg_get_get_catalog_file()
         fi
 
         if [[ ${cur} == -* ]]; then
-            local opts="-c -d -D -f -i -l -s -S -u -U -v"
-            COMPREPLY=($(compgen -W "${opts}" -- ${cur}))
+            local -a opts=(-c -d -D -f -i -l -s -S -u -U -v)
+            COMPREPLY=($(compgen -W '${opts[@]}' -- ${cur}))
             return
         fi
 
-        local commands="available describe download install list \
-        updatecatalog upgrade"
-        COMPREPLY=($(compgen -W "${commands}" -- ${cur}))
+        local -a commands=(
+            available describe download install list updatecatalog upgrade
+        )
+        COMPREPLY=($(compgen -W '${commands[@]}' -- ${cur}))
     } &&
         complete -F _pkg_get pkg-get
 

completions/pkgadd

@@ -36,8 +36,8 @@ _pkgadd()
 
         *)
             if [[ ${cur} == -* ]]; then
-                local opts="-a -A -d -k -n -M -P -r -R -s -v -V -x"
-                COMPREPLY=($(compgen -W "${opts}" -- ${cur}))
+                local -a opts=(-a -A -d -k -n -M -P -r -R -s -v -V -x)
+                COMPREPLY=($(compgen -W '${opts[@]}' -- ${cur}))
             else
                 local pkginst_list
                 if [[ -d $device ]]; then

completions/rpm

@@ -118,45 +118,49 @@ _rpm()
     $split && return
 
     # options common to all modes
-    local opts="--define= --eval= --macros= --nodigest --nosignature --rcfile=
-        --quiet --pipe --verbose"
+    local -a opts=(
+        --define= --eval= --macros= --nodigest --nosignature --rcfile= --quiet
+        --pipe --verbose
+    )
 
     case ${words[1]} in
         -[iFU]* | --install | --freshen | --upgrade)
             if [[ $cur == -* ]]; then
-                COMPREPLY=($(compgen -W "$opts --percent --force --test
+                COMPREPLY=($(compgen -W '${opts[@]} --percent --force --test
                 --replacepkgs --replacefiles --root --excludedocs --includedocs
                 --noscripts --ignorearch --dbpath --prefix= --ignoreos --nodeps
                 --allfiles --ftpproxy --ftpport --justdb --httpproxy --httpport
                 --noorder --relocate= --badreloc --notriggers --excludepath=
                 --ignoresize --oldpackage --queryformat --repackage
-                --nosuggests" -- "$cur"))
+                --nosuggests' -- "$cur"))
             else
                 _filedir '[rs]pm'
             fi
             ;;
         -e | --erase)
             if [[ $cur == -* ]]; then
-                COMPREPLY=($(compgen -W "$opts --allmatches --noscripts
-                    --notriggers --nodeps --test --repackage" -- "$cur"))
+                COMPREPLY=($(compgen -W '${opts[@]} --allmatches --noscripts
+                    --notriggers --nodeps --test --repackage' -- "$cur"))
             else
                 _rpm_installed_packages $1
             fi
             ;;
         -q* | --query)
             # options common to all query types
-            opts+=" --changelog --configfiles --conflicts --docfiles --dump
+            opts+=(
+                --changelog --configfiles --conflicts --docfiles --dump
                 --enhances --filesbypkg --filecaps --fileclass --filecolor
                 --fileprovide --filerequire --filesbypkg --info --list
                 --obsoletes --pipe --provides --queryformat= --requires
                 --scripts --suggests --triggers --xml --recommends
-                --supplements --filetriggers --licensefiles"
+                --supplements --filetriggers --licensefiles
+            )
 
             if [[ ${words[*]} == *\ -@(*([^ -])f|-file )* ]]; then
                 # -qf completion
                 if [[ $cur == -* ]]; then
-                    COMPREPLY=($(compgen -W "$opts --dbpath --fscontext
-                        --last --root --state" -- "$cur"))
+                    COMPREPLY=($(compgen -W '${opts[@]} --dbpath --fscontext
+                        --last --root --state' -- "$cur"))
                 else
                     _filedir
                 fi
@@ -166,41 +170,40 @@ _rpm()
             elif [[ ${words[*]} == *\ -@(*([^ -])p|-package )* ]]; then
                 # -qp; uninstalled package completion
                 if [[ $cur == -* ]]; then
-                    COMPREPLY=($(compgen -W "$opts --ftpport --ftpproxy
-                        --httpport --httpproxy --nomanifest" -- "$cur"))
+                    COMPREPLY=($(compgen -W '${opts[@]} --ftpport --ftpproxy
+                        --httpport --httpproxy --nomanifest' -- "$cur"))
                 else
                     _filedir '[rs]pm'
                 fi
             else
                 # -q; installed package completion
                 if [[ $cur == -* ]]; then
-                    COMPREPLY=($(compgen -W "$opts --all --file --fileid
+                    COMPREPLY=($(compgen -W '${opts[@]} --all --file --fileid
                         --dbpath --fscontext --ftswalk --group --hdrid --last
                         --package --pkgid --root= --specfile --state
                         --triggeredby --whatenhances --whatprovides
                         --whatrecommends --whatrequires --whatsuggests
-                        --whatsupplements" \
-                        -- "$cur"))
+                        --whatsupplements' -- "$cur"))
                 elif [[ ${words[*]} != *\ -@(*([^ -])a|-all )* ]]; then
                     _rpm_installed_packages $1
                 fi
             fi
             ;;
         -K* | --checksig)
             if [[ $cur == -* ]]; then
-                COMPREPLY=($(compgen -W "$opts --nopgp --nogpg --nomd5" \
+                COMPREPLY=($(compgen -W '${opts[@]} --nopgp --nogpg --nomd5' \
                     -- "$cur"))
             else
                 _filedir '[rs]pm'
             fi
             ;;
         -[Vy]* | --verify)
             if [[ $cur == -* ]]; then
-                COMPREPLY=($(compgen -W "$opts --root= --dbpath --nodeps
+                COMPREPLY=($(compgen -W '${opts[@]} --root= --dbpath --nodeps
                     --nogroup --nolinkto --nomode --nomtime --nordev --nouser
                     --nofiles --noscripts --nomd5 --querytags --specfile
                     --whatenhances --whatprovides --whatrecommends
-                    --whatrequires --whatsuggests --whatsupplements" \
+                    --whatrequires --whatsuggests --whatsupplements' \
                     -- "$cur"))
             # check whether we're doing file completion
             elif [[ ${words[*]} == *\ -@(*([^ -])f|-file )* ]]; then