diff options
author | Wynn Wolf Arbor | 2020-05-13 15:22:34 +0200 |
---|---|---|
committer | Wynn Wolf Arbor | 2020-05-13 15:22:34 +0200 |
commit | 10b1d754142254727c6f47259c927f10681656d0 (patch) | |
tree | 67c7ab1b0b56ef24c50cdffa0d320718084c62a8 | |
parent | 4579c8411ef7dae1ac871555a19f1bfbfae5b54c (diff) | |
download | bosun-10b1d754142254727c6f47259c927f10681656d0.tar.gz |
Simplify handling of default actions
Instead of using a catchall pattern for `case`, assign a default value
to the variable in question. This helps reduce the amount of special
cases - especially considering the addition of better error reporting in
an upcoming commit.
-rwxr-xr-x | bosun | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -34,12 +34,11 @@ rebuild() { } list() { - subcmd="$1" + subcmd="${1:-active}" case $subcmd in active) list_active;; all) list_all;; available) list_available;; - *) list_active;; esac } @@ -64,7 +63,7 @@ list_available() { rm -f "$tmp" } -cmd="$1" +cmd="${1:-list}" test $# -gt 0 && shift case $cmd in add) add "$@";; @@ -72,5 +71,4 @@ case $cmd in list) list "$@";; rebuild) rebuild "$@";; remove) remove "$@";; - *) list_active;; esac |