Compact option processing.

Most of the options are one-liners, and the shift can be factored out of
the case expression entirely.
---
 dracut |   31 +++++++++----------------------
 1 files changed, 9 insertions(+), 22 deletions(-)

diff --git a/dracut b/dracut
index 0fe0282..82cf9e5 100755
--- a/dracut
+++ b/dracut
@@ -10,30 +10,17 @@
 
 [ -f /etc/dracut.conf ] && . /etc/dracut.conf
 
-while [ $# -gt 0 ]; do
+while (($# > 0)); do
     case $1 in
-       -f|--force)
-           force=yes
-           shift
-           ;;
-       -h|--help)
-           echo "Usage: $0 [-f] <initramfs> <kernel-version>"
-           exit 1
-           ;;
-       -v|--verbose)
-           set -x
-           shift
-           ;;
-       -l|--local)
-           allowlocal="yes"
-           shift
-           ;;
-       --allow-missing)
-           shift
-           ;;
-       *)
-           break
+       -f|--force) force=yes;;
+       -h|--help) echo "Usage: $0 [-f] <initramfs> <kernel-version>"
+           exit 1 ;;
+       -v|--verbose) set -x;;
+       -l|--local) allowlocal="yes" ;;
+       --allow-missing) : ;;
+       *) break ;;
     esac
+    shift
 done
 
 if [ -n "$2" ]; then
-- 
1.6.0.6

--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to