Consistently using case instead of if for conditionals in script boilerplate.
authorDaniel Baumann <daniel@debian.org>
Mon, 13 Aug 2012 23:20:28 +0000 (01:20 +0200)
committerDaniel Baumann <daniel@debian.org>
Mon, 13 Aug 2012 23:20:28 +0000 (01:20 +0200)
scripts/boot/0110-debug
scripts/boot/0120-read-only
scripts/boot/3010-verify-checksums
scripts/boot/3020-swapon

index a74dcb7..f223e93 100755 (executable)
@@ -13,10 +13,15 @@ Debug ()
                esac
        done
 
-       if [ "${LIVE_DEBUG}" != "true" ]
-       then
-               return 0
-       fi
+       case "${LIVE_DEBUG}" in
+               true)
+                       ;;
 
+               *)
+                       return 0
+                       ;;
+       esac
+
+       # Write the trace output
        set -x
 }
index c05c162..859f771 100755 (executable)
@@ -18,10 +18,14 @@ Read_only ()
                esac
        done
 
-       if [ "${LIVE_READ_ONLY}" != "true" ]
-       then
-               return 0
-       fi
+       case "${LIVE_READ_ONLY}" in
+               true)
+                       ;;
+
+               *)
+                       return 0
+                       ;;
+       esac
 
        # Marking some block devices as read-only to ensure that nothing
        # gets written as linux still writes to 'only' read-only mounted filesystems.
index 3649e04..8ccd663 100755 (executable)
@@ -18,10 +18,14 @@ Verify_checksums ()
                esac
        done
 
-       if [ "${LIVE_VERIFY_CHECKSUMS}" != "true" ]
-       then
-               return 0
-       fi
+       case "${LIVE_VERIFY_CHECKSUMS}" in
+               true)
+                       ;;
+
+               *)
+                       return 0
+                       ;;
+       esac
 
        _MOUNTPOINT="${1}"
 
index 82e998a..7541c26 100755 (executable)
@@ -18,10 +18,14 @@ Swap ()
                esac
        done
 
-       if [ "${LIVE_SWAP}" != "true" ]
-       then
-               return 0
-       fi
+       case "${LIVE_SWAP}" in
+               true)
+                       ;;
+
+               *)
+                       return 0
+                       ;;
+       esac
 
        LIVE_SWAP_DEVICES="${LIVE_SWAP_DEVICES:-/dev/sd* /dev/vd*}"