okra pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=d4bc101866c09188bc90d68e3cd9f72d294c3606

commit d4bc101866c09188bc90d68e3cd9f72d294c3606
Author: Stephen 'Okra' Houston <smhousto...@gmail.com>
Date:   Wed Jul 12 13:32:56 2017 -0500

    Cpuclock: Clean up some governor handling code.
---
 src/modules/sysinfo/cpuclock/cpuclock.c        | 3 ++-
 src/modules/sysinfo/cpuclock/cpuclock_config.c | 8 +++++---
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/modules/sysinfo/cpuclock/cpuclock.c 
b/src/modules/sysinfo/cpuclock/cpuclock.c
index 71dab70f6..0172a5e79 100644
--- a/src/modules/sysinfo/cpuclock/cpuclock.c
+++ b/src/modules/sysinfo/cpuclock/cpuclock.c
@@ -67,7 +67,8 @@ _cpuclock_set_thread_governor(void *data, Ecore_Thread *th 
EINA_UNUSED)
 {
    const char *governor = data;
 
-   _cpuclock_sysfs_setall("scaling_governor", governor);
+   if (_cpuclock_sysfs_setall("scaling_governor", governor) == 0)
+     return;
    if (!strcmp(governor, "ondemand"))
      _cpuclock_sysfs_set("ondemand/ignore_nice_load", "0");
    else if (!strcmp(governor, "conservative"))
diff --git a/src/modules/sysinfo/cpuclock/cpuclock_config.c 
b/src/modules/sysinfo/cpuclock/cpuclock_config.c
index 995258370..255b10104 100644
--- a/src/modules/sysinfo/cpuclock/cpuclock_config.c
+++ b/src/modules/sysinfo/cpuclock/cpuclock_config.c
@@ -154,7 +154,7 @@ _powersave_changed(void *data, Evas_Object *obj, void 
*event_info EINA_UNUSED)
 {
    Cpuclock_Config *cc = data;
    Instance *inst = cc->inst;
-   const char *value = elm_object_text_get(obj);
+   const char *value = evas_object_data_get(obj, "governor");
 
    if (value)
      eina_stringshare_replace(&inst->cfg->cpuclock.powersave_governor, value);
@@ -167,7 +167,7 @@ _governor_changed(void *data, Evas_Object *obj, void 
*event_info EINA_UNUSED)
 {
    Cpuclock_Config *cc = data;
    Instance *inst = cc->inst;
-   const char *value = elm_object_text_get(obj);
+   const char *value = evas_object_data_get(obj, "governor");
 
    if (value)
      {
@@ -403,7 +403,8 @@ cpuclock_configure(Instance *inst)
           elm_object_text_set(o, _("Maximum Speed"));
         else
           elm_object_text_set(o, l->data);
-        elm_box_pack_end(box, o);
+       evas_object_data_set(o, "governor", (const char *)l->data);
+       elm_box_pack_end(box, o);
         evas_object_smart_callback_add(o, "changed", _governor_changed, cc);
         evas_object_show(o);
 
@@ -472,6 +473,7 @@ cpuclock_configure(Instance *inst)
         else
           elm_object_text_set(o, l->data);
         elm_object_disabled_set(o, inst->cfg->cpuclock.auto_powersave);
+       evas_object_data_set(o, "governor", l->data);
        elm_box_pack_end(box, o);
         evas_object_smart_callback_add(o, "changed", _powersave_changed, cc);
         evas_object_show(o);

-- 


Reply via email to