Simplify cmake targets

- Don't use for loop
- Use reduce filtering commands

src/_cmake

@@ -165,25 +165,16 @@ _cmake_presets() {
 # --------------
 (( $+functions[_cmake_targets] )) ||
 _cmake_targets() {
-  local -a targets
+  local dir="$1"
-  local i
+  local -a targets=()
-  if [ -f $1/Makefile ]
+  if [[ -f "${dir}/Makefile" ]]; then
-  then
     # `make help` doesn't work for Makefiles in general, but for CMake generated Makefiles it does.
-    i=1
+    targets=(${(f)"$(make -f $dir/Makefile help | awk '/^\.\.\./ { print $2 }')"})
-    for target in $(make -f $1/Makefile help | \grep -e "\.\.\." | sed "s/\.\.\. //" | sed "s/ (the default.*//") ; do
+  elif [[ -f "${dir}/build.ninja" ]]; then
-      targets[$i]=$target
-      (( i = $i + 1 ))
-    done
-  elif [ -f $1/build.ninja ]
-  then
     # `ninja help` doesn't seem to be the list of targets we're interested in
-    i=1
+    targets=(${(f)"$(ninja -C $dir -t targets all 2>/dev/null | awk -F: '{print $1}' )"})
-    for target in $(ninja -C $1 -t targets all 2&>/dev/null | awk -F: '{print $1}') ; do
-      targets[$i]="$target"
-      (( i++ ))
-    done
   fi
+
   _describe 'build targets' targets
 }