refactor: switch from if "$var" to if [[ $var ]]

Author: akinomyoga

bash_completion

@@ -339,12 +339,12 @@ _comp_expand_glob()
 #   generated.
 _comp_split()
 {
-    local _append=false IFS=$' \t\n'
+    local _append="" IFS=$' \t\n'
 
     local OPTIND=1 OPTARG="" OPTERR=0 _opt
     while getopts ':alF:' _opt "$@"; do
         case $_opt in
-            a) _append=true ;;
+            a) _append=set ;;
             l) IFS=$'\n' ;;
             F) IFS=$OPTARG ;;
             *)
@@ -367,7 +367,7 @@ _comp_split()
     set -o noglob
 
     local _old_size _new_size
-    if "$_append"; then
+    if [[ $_append ]]; then
         eval "$1+=()" # in case $1 is unset
         eval "_old_size=\${#$1[@]}"
         eval "$1+=(\$2)"
@@ -409,13 +409,13 @@ _comp_split()
 #   array instead.
 _comp_compgen()
 {
-    local _append=false IFS=$' \t\n'
+    local _append="" IFS=$' \t\n'
     local -a _split_options=(-l)
 
     local OPTIND=1 OPTARG="" OPTERR=0 _opt
     while getopts ':alF:' _opt "$@"; do
         case $_opt in
-            a) _append=true _split_options+=(-a) ;;
+            a) _append=set _split_options+=(-a) ;;
             l) IFS=$'\n' ;;
             F) IFS=$OPTARG ;;
             *)
@@ -447,7 +447,7 @@ _comp_compgen()
     local _result
     _result=$(compgen "${@:2}") || {
         local _status=$?
-        if "$_append"; then
+        if [[ $_append ]]; then
             # make sure existence of variable
             eval -- "$1+=()"
         else

completions/_adb

@@ -25,16 +25,16 @@ _comp_cmd_adb()
             ;;
     esac
 
-    local cmd has_cmd=false i
+    local cmd has_cmd="" i
     for ((i = 1; i < cword; i++)); do
         if [[ ${words[i]} != -* && ${words[i - 1]} != -[sp] ]]; then
             cmd="${words[i]}"
-            has_cmd=true
+            has_cmd=set
             break
         fi
     done
 
-    if ! "$has_cmd"; then
+    if [[ ! $has_cmd ]]; then
         local tmp=()
         if [[ ! $cur || $cur == -* ]]; then
             tmp+=($(compgen -W '$(_parse_help "$1" help)' -- "$cur"))

completions/_chsh

@@ -8,13 +8,13 @@ _comp_cmd_chsh()
     local cur prev words cword comp_args
     _comp_initialize -- "$@" || return
 
-    local word chroot="" has_chroot=false
+    local word chroot="" has_chroot=""
     for word in "${words[@]}"; do
-        if "$has_chroot"; then
+        if [[ $has_chroot ]]; then
             chroot=$word
             break
         fi
-        [[ $word != -@(R|-root) ]] || has_chroot=true
+        [[ $word != -@(R|-root) ]] || has_chroot=set
     done
 
     case $prev in

completions/_mount.linux

@@ -8,23 +8,23 @@ _comp_cmd_mount()
     local cur prev words cword comp_args
     _comp_initialize -n =: -- "$@" || return
 
-    local split=false
+    local split=""
     case "$prev" in
         -t | --types)
             # find /lib/modules/$(uname -r)/ -type f -path '*/fs/*.ko' -printf '%f\n' | cut -d. -f1
             # FIXME: no<fstype>
             if [[ $cur == ?*,* ]]; then
                 prev="${cur%,*}"
                 cur="${cur##*,}"
-                split=true
+                split=set
             fi
             COMPREPLY=($(compgen -W 'auto adfs affs autofs btrfs cifs coda
                 cramfs davfs debugfs devpts efs ext2 ext3 ext4 fuse hfs hfsplus
                 hpfs iso9660 jffs2 jfs minix msdos ncpfs nfs nfs4 ntfs ntfs-3g
                 proc qnx4 ramfs reiserfs romfs squashfs smbfs sysv tmpfs ubifs
                 udf ufs umsdos usbfs vfat xfs' -- "$cur"))
             _fstypes
-            $split && COMPREPLY=(${COMPREPLY[@]/#/$prev,})
+            [[ $split ]] && COMPREPLY=(${COMPREPLY[@]/#/$prev,})
             return
             ;;
         --bind | -B | --rbind | -R)
@@ -74,7 +74,7 @@ _comp_cmd_mount()
             if [[ $cur == ?*,* ]]; then
                 prev="${cur%,*}"
                 cur="${cur##*,}"
-                split=true
+                split=set
             fi
             # no completion if $cur is opt=smth
             [[ $cur == *=* ]] && return
@@ -206,7 +206,7 @@ _comp_cmd_mount()
             esac
             # COMP_WORDBREAKS is a real pain in the ass
             prev="${prev##*["$COMP_WORDBREAKS"]}"
-            $split && COMPREPLY=(${COMPREPLY[@]/#/"$prev,"})
+            [[ $split ]] && COMPREPLY=(${COMPREPLY[@]/#/"$prev,"})
             [[ ${COMPREPLY-} == *= ]] && compopt -o nospace
             return
             ;;

completions/_slackpkg

@@ -9,11 +9,11 @@ _comp_cmd_slackpkg()
     local cur prev words cword comp_args
     _comp_initialize -n = -- "$@" || return
 
-    local split=false
+    local split=""
     if [[ $cur == -?*=* ]]; then
         prev="${cur%%?(\\)=*}"
         cur="${cur#*=}"
-        split=true
+        split=set
     fi
 
     case "$prev" in
@@ -33,7 +33,7 @@ _comp_cmd_slackpkg()
             ;;
     esac
 
-    $split && return
+    [[ $split ]] && return
 
     if [[ $cur == -* ]]; then
         compopt -o nospace

completions/_udevadm

@@ -8,11 +8,11 @@ _comp_cmd_udevadm()
     local cur prev words cword split comp_args
     _comp_initialize -s -- "$@" || return
 
-    local i udevcmd has_udevcmd=false
+    local i udevcmd has_udevcmd=""
     for ((i = 1; i < cword; i++)); do
         if [[ ${words[i]} != -* ]]; then
             udevcmd=${words[i]}
-            has_udevcmd=true
+            has_udevcmd=set
             break
         fi
     done
@@ -54,7 +54,7 @@ _comp_cmd_udevadm()
 
     $split && return
 
-    if ! "$has_udevcmd"; then
+    if [[ ! $has_udevcmd ]]; then
         case $cur in
             -*)
                 COMPREPLY=($(compgen -W '--help --version --debug' -- "$cur"))

completions/_umount.linux

@@ -101,19 +101,19 @@ _comp_cmd_umount()
     case "$prev" in
         -t)
             # FIXME: no<fstype>
-            local split=false
+            local split=""
             if [[ $cur == ?*,* ]]; then
                 prev="${cur%,*}"
                 cur="${cur##*,}"
-                split=true
+                split=set
             fi
             COMPREPLY=($(compgen -W 'adfs affs autofs btrfs cifs coda
                 cramfs debugfs devpts efs ext2 ext3 ext4 fuse hfs hfsplus hpfs
                 iso9660 jfs minix msdos ncpfs nfs nfs4 ntfs ntfs-3g proc qnx4
                 ramfs reiserfs romfs squashfs smbfs sysv tmpfs ubifs udf ufs
                 umsdos usbfs vfat xfs' -- "$cur"))
             _fstypes
-            $split && COMPREPLY=(${COMPREPLY[@]/#/$prev,})
+            [[ $split ]] && COMPREPLY=(${COMPREPLY[@]/#/$prev,})
             return
             ;;
         -O)

completions/carton

@@ -18,19 +18,19 @@ _carton()
     local cur prev words cword split comp_args
     _comp_initialize -s -- "$@" || return
 
-    local i command="" has_command=false
+    local i command="" has_command=""
     for ((i = 1; i < cword; i++)); do
         case ${words[i]} in
             -*) ;;
             *)
                 command=${words[i]}
-                has_command=true
+                has_command=set
                 break
                 ;;
         esac
     done
 
-    if ! "$has_command"; then
+    if [[ ! $has_command ]]; then
         _carton_commands "$1"
         return
     fi
@@ -40,7 +40,7 @@ _carton()
             return
             ;;
         --help | -h)
-            "$has_command" || _carton_commands "$1"
+            [[ $has_command ]] || _carton_commands "$1"
             return
             ;;
         --cpanfile)

completions/cvs

@@ -53,7 +53,7 @@ _cvs()
     local cur prev words cword comp_args
     _comp_initialize -n : -- "$@" || return
 
-    local count mode="" i cvsroot="" has_cvsroot=false cvsroots pwd
+    local count mode="" i cvsroot="" has_cvsroot="" cvsroots pwd
     local -a flags files entries changed newremoved
 
     local noargopts='!(-*|*[d]*)'
@@ -74,7 +74,7 @@ _cvs()
                 -${noargopts}d)
                     mode=cvsroot
                     cvsroot=${words[count + 1]}
-                    has_cvsroot=true
+                    has_cvsroot=set
                     ;;
                 add | ad | new)
                     mode=add
@@ -232,7 +232,7 @@ _cvs()
             esac
 
             if [[ $cur != -* ]]; then
-                ! "$has_cvsroot" && cvsroot=${CVSROOT-}
+                [[ ! $has_cvsroot ]] && cvsroot=${CVSROOT-}
                 COMPREPLY=($(cvs -d "$cvsroot" co -c 2>/dev/null |
                     awk '{print $1}'))
                 ((${#COMPREPLY[@]})) &&
@@ -317,7 +317,7 @@ _cvs()
             esac
 
             if [[ $cur != -* ]]; then
-                ! "$has_cvsroot" && cvsroot=${CVSROOT-}
+                [[ ! $has_cvsroot ]] && cvsroot=${CVSROOT-}
                 COMPREPLY=($(cvs -d "$cvsroot" co -c | awk '{print $1}'))
                 ((${#COMPREPLY[@]})) &&
                     COMPREPLY=($(compgen -W '"${COMPREPLY[@]}"' -- "$cur"))
@@ -339,7 +339,7 @@ _cvs()
 
             if [[ $cur != -* ]]; then
                 # starts with same algorithm as checkout
-                ! "$has_cvsroot" && cvsroot=${CVSROOT-}
+                [[ ! $has_cvsroot ]] && cvsroot=${CVSROOT-}
                 local prefix=${cur%/*}
                 if [[ -r ${cvsroot}/${prefix} ]]; then
                     _cvs_modules

completions/dmypy

@@ -15,11 +15,11 @@ _comp_cmd_dmypy()
             ;;
     esac
 
-    local cmd has_cmd=false i
+    local cmd has_cmd="" i
     for ((i = 1; i < cword; i++)); do
         if [[ ${words[i]} != -* && ${words[i - 1]} != --status-file ]]; then
             cmd=${words[i]}
-            has_cmd=true
+            has_cmd=set
             break
         fi
     done
@@ -36,7 +36,7 @@ _comp_cmd_dmypy()
         return
     fi
 
-    if ! "$has_cmd"; then
+    if [[ ! $has_cmd ]]; then
         local cmds=$("$1" --help 2>&1 |
             command sed -ne '/positional arguments/{p;n;p;q;}' |
             command sed -ne 's/{\(.*\)}/\1/p')