[Pm-utils] [PATCH 6/6] Modified acpi_flag setting to use arithmetic expansion.

Victor Lowther victor.lowther at gmail.com
Wed Feb 27 18:29:16 PST 2008


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



More information about the Pm-utils mailing list