diff options
| author | Joseph Herlant <aerostitch@debian.org> | 2019-08-30 12:00:54 -0700 |
|---|---|---|
| committer | jkoan <jkoan@users.noreply.github.com> | 2019-08-31 14:10:15 +0200 |
| commit | fbd0b044053f9158f106ff38e0c02c71a7474122 (patch) | |
| tree | 3cfa821cac2e4648600e855fdfa7717bd8f915fc /navit/startonce.sh | |
| parent | bcc10a828e53153b5bda731f6fbb3f0f6fb4aec1 (diff) | |
| download | navit-fbd0b044053f9158f106ff38e0c02c71a7474122.tar.gz | |
Refactoring:tools: Check exit code directly with e.g. "if mycmd;", not indirectly with $?
Diffstat (limited to 'navit/startonce.sh')
| -rwxr-xr-x | navit/startonce.sh | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/navit/startonce.sh b/navit/startonce.sh index 3e27d77f0..04220b33b 100755 --- a/navit/startonce.sh +++ b/navit/startonce.sh @@ -21,9 +21,8 @@ NAVIT="./navit" function check_wmctrl() { - which wmctrl > /dev/null - - if [ $? -ne 0 ] ; then + if ! which wmctrl > /dev/null + then echo "I need the 'wmctrl' program. Exit." exit 1 fi @@ -39,9 +38,8 @@ function start_navit() pid=$! - echo -n "$pid" > $PIDFILE - - if [ $? -eq 0 ] ; then + if ! echo -n "$pid" > $PIDFILE + then echo "Started navit with PID $pid." else kill $pid @@ -59,12 +57,12 @@ function check_navit() { if [ -f $PIDFILE ] ; then pid=$(cat $PIDFILE) - kill -0 $pid 2>/dev/null - if [ $? -eq 0 ] ; then - echo "Bringing Navit to front" + if ! kill -0 $pid 2>/dev/null + then + echo "Bringing Navit to front" - winid=$(wmctrl -l -p | grep -e "^[^:blank:]*[:blank:]*[^:blank:]*[:blank:]*$pid[:blank:]*" | sed 's/ .*//') - wmctrl -i -R $winid + winid=$(wmctrl -l -p | grep -e "^[^:blank:]*[:blank:]*[^:blank:]*[:blank:]*$pid[:blank:]*" | sed 's/ .*//') + wmctrl -i -R $winid exit 0 fi |