vmadm alias completion

gistfile1.txt

    ]] && [[ ${COMP_WORDS[COMP_CWORD-2]} == "vmadm" ]]; then

  vms_uuids=$(zoneadm list -cp | grep -v ':global:' | cut -d':' -f5 | \
      sort | uniq)
	vms_aliases=$(vmadm list -Ho alias | sort -u)
	gen=$(compgen -W "${vms_aliases}" -- ${cur})
	if [ "$(echo $gen | wc -w)" -eq 1 ]; then 
		COMPREPLY=$(vmadm list -Ho uuid alias=$gen)
	else
        	COMPREPLY=( $(compgen -W "${vms_uuids} ${vms_aliases}" -- ${cur}) )
	fi

You can get rid of wc by using an array.

[root@buglets ~]# typeset -a vms_aliases
[root@buglets ~]# vms_aliases=( $(vmadm list -Ho alias | sort -u) )
[root@buglets ~]# echo ${#vms_aliases[@]}
5
[root@buglets ~]# echo ${vms_aliases[@]}
docker-redis joyent log test test1710

This also has issues with multiple zones having the same alias and aliases with spaces (assuming that's a thing).