Fix edge glow crash

Domi04151309 · Dec 30, 2023 · 5aa7584 · 5aa7584
1 parent 87375b1
commit 5aa7584
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 12 deletions.
diff --git a/app/src/main/java/io/github/domi04151309/alwayson/actions/alwayson/EdgeGlowThread.kt b/app/src/main/java/io/github/domi04151309/alwayson/actions/alwayson/EdgeGlowThread.kt
@@ -11,21 +11,20 @@ class EdgeGlowThread(
     private val activity: Activity,
     private val background: TransitionDrawable?,
 ) : Thread() {
-    private val transitionTime =
-        P.getPreferences(activity).getInt(
-            P.EDGE_GLOW_DURATION,
-            P.EDGE_GLOW_DURATION_DEFAULT,
-        )
-    private val transitionDelay =
-        P.getPreferences(activity).getInt(
-            P.EDGE_GLOW_DELAY,
-            P.EDGE_GLOW_DELAY_DEFAULT,
-        )
-
     @JvmField
     internal var notificationAvailable: Boolean = false
 
     override fun run() {
+        val transitionTime =
+            P.getPreferences(activity).getInt(
+                P.EDGE_GLOW_DURATION,
+                P.EDGE_GLOW_DURATION_DEFAULT,
+            )
+        val transitionDelay =
+            P.getPreferences(activity).getInt(
+                P.EDGE_GLOW_DELAY,
+                P.EDGE_GLOW_DELAY_DEFAULT,
+            )
         try {
             while (!isInterrupted) {
                 if (notificationAvailable) {

diff --git a/app/src/main/java/io/github/domi04151309/alwayson/activities/LAFBrightnessActivity.kt b/app/src/main/java/io/github/domi04151309/alwayson/activities/LAFBrightnessActivity.kt
@@ -8,6 +8,10 @@ import io.github.domi04151309.alwayson.actions.alwayson.AlwaysOn
 import io.github.domi04151309.alwayson.helpers.P
 
 class LAFBrightnessActivity : BaseActivity() {
+    companion object {
+        private const val FULL_BRIGHTNESS = 100.0
+    }
+
     private lateinit var brightnessSwitch: SwitchCompat
     private lateinit var seekBar: SeekBar
 
@@ -27,7 +31,7 @@ class LAFBrightnessActivity : BaseActivity() {
                 ) {
                     // Turning this into a single statement will not work!
                     val attributes = window.attributes
-                    attributes.screenBrightness = (progress / 100.0).toFloat()
+                    attributes.screenBrightness = (progress / FULL_BRIGHTNESS).toFloat()
                     window.attributes = attributes
                 }