This gets rid of an ugly if statement and makes the code more legible.
---
 pm/sleep.d/99video |   14 ++++----------
 1 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/pm/sleep.d/99video b/pm/sleep.d/99video
index 4cb62dc..54fdb66 100755
--- a/pm/sleep.d/99video
+++ b/pm/sleep.d/99video
@@ -53,16 +53,10 @@ resume_fbcon()
 suspend_video()
 {
        # 0=nothing, 1=s3_bios, 2=s3_mode, 3=both
-       if [ "${DISPLAY_QUIRK_S3_BIOS}" = "true" -a \
-               "${DISPLAY_QUIRK_S3_MODE}" = "true" ]; then
-               sysctl -w kernel.acpi_video_flags=3
-       elif [ "${DISPLAY_QUIRK_S3_BIOS}" = "true" ]; then
-               sysctl -w kernel.acpi_video_flags=1
-       elif [ "${DISPLAY_QUIRK_S3_MODE}" = "true" ]; then
-               sysctl -w kernel.acpi_video_flags=2
-       else
-               sysctl -w kernel.acpi_video_flags=0
-       fi
+       local acpi_flag=0
+       [ "${DISPLAY_QUIRK_S3_BIOS}" = "true" ] && acpi_flag=$(($acpi_flag + 1))
+       [ "${DISPLAY_QUIRK_S3_MODE}" = "true" ] && acpi_flag=$(($acpi_flag + 2))
+       sysctl -w kernel.acpi_video_flags=$acpi_flag
 
        # We might need to do one or many of these quirks
        if [ "${DISPLAY_QUIRK_RADEON_OFF}" = "true" ]; then
-- 
1.5.4.3

_______________________________________________
Pm-utils mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pm-utils

Reply via email to