From 10b1d754142254727c6f47259c927f10681656d0 Mon Sep 17 00:00:00 2001 From: Wynn Wolf Arbor Date: Wed, 13 May 2020 15:22:34 +0200 Subject: 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. --- bosun | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/bosun b/bosun index b462b6b..c195176 100755 --- a/bosun +++ b/bosun @@ -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 -- cgit v1.2.3-2-gb3c3