use separate variable for backlight animator value

using e_bl_val here is wrong now since this needs to calculate against the starting and ending backlight values, neither of which are represented accurately by e_bl_val in some cases.

this fixes the gradually decaying backlight which occurs over repeated screen blanks

probably fixes T1152

diff --git a/src/bin/e_backlight.c b/src/bin/e_backlight.c
index 5256bb6..130de88 100644 (file)
--- a/src/bin/e_backlight.c
+++ b/src/bin/e_backlight.c
@@ -13,6 +13,7 @@
 
 EINTERN double e_bl_val = 1.0;
 static double bl_animval = 1.0;
+static double bl_anim_toval = 1.0;
 static int sysmode = MODE_NONE;
 static Ecore_Animator *bl_anim = NULL;
 static Eina_List *bl_devs = NULL;
@@ -144,6 +145,7 @@ e_backlight_level_set(E_Zone *zone, double val, double tim)
    E_FREE_FUNC(bl_anim, ecore_animator_del);
    bl_anim = ecore_animator_timeline_add(tim, _bl_anim, zone);
    bl_animval = bl_now;
+   bl_anim_toval = val;
 }
 
 EAPI double
@@ -317,7 +319,7 @@ _bl_anim(void *data, double pos)
 
    // FIXME: if zone is deleted while anim going... bad things.
    pos = ecore_animator_pos_map(pos, ECORE_POS_MAP_DECELERATE, 0.0, 0.0);
-   v = (bl_animval * (1.0 - pos)) + (e_bl_val * pos);
+   v = (bl_animval * (1.0 - pos)) + (bl_anim_toval * pos);
    _e_backlight_set(zone, v);
    if (pos >= 1.0)
      {