Merge pull request #276 from KaushikMalapati/shellcheck-daq_waitwin

Shellcheck daq waitwin

Author: KaushikMalapati

scripts/daq_waitwin

@@ -17,14 +17,14 @@ source /reg/g/pcds/setup/pathmunge.sh
 ldpathmunge /usr/local/lib
 
 
-if [[ $# > 0 ]]; then
-    SEARCHOPT='--onlyvisible --maxdepth 2 --limit 1 --all --desktop '$1' --sync '
-else 
-    SEARCHOPT='--onlyvisible --maxdepth 2 --limit 1 --all --sync '
+if [[ $# -gt 0 ]]; then
+    SEARCHOPT=(--onlyvisible --maxdepth 2 --limit 1 --all --desktop "$1" --sync )
+else
+    SEARCHOPT=(--onlyvisible --maxdepth 2 --limit 1 --all --sync )
 fi
-proc=`xdotool search $SEARCHOPT --name 'ProcStat'`
-ami=`xdotool search $SEARCHOPT --name 'DAQ Online Monitoring'`
-env=`xdotool search $SEARCHOPT --name 'Env'`
-crwin=`xdotool search $SEARCHOPT --name 'DAQ Control'` 
-echo $proc $ami $env $crwin
-exit
+proc=$(xdotool search "${SEARCHOPT[@]}" --name 'ProcStat')
+ami=$(xdotool search "${SEARCHOPT[@]}" --name 'DAQ Online Monitoring')
+env=$(xdotool search "${SEARCHOPT[@]}" --name 'Env')
+crwin=$(xdotool search "${SEARCHOPT[@]}" --name 'DAQ Control')
+echo "$proc $ami $env $crwin"
+exit