netstar pushed a commit to branch master.

http://git.enlightenment.org/apps/evisum.git/commit/?id=83a33bd99160ba514a27fcd2d60b0307866a03ae

commit 83a33bd99160ba514a27fcd2d60b0307866a03ae
Author: Alastair Poole <nets...@gmail.com>
Date:   Tue Dec 22 13:47:41 2020 +0000

    sensors: Kernel doesn't cache thermal.
    
    Linux doesn't cache thermal readings, select one sensor else we
    can block waiting to read from N sensors.
    
    Needs more fixing.
---
 src/bin/system/machine.h             |  12 +
 src/bin/system/machine/sensors.bogox |  73 ++++-
 src/bin/ui/ui.h                      |  10 +-
 src/bin/ui/ui_sensors.c              | 582 ++++++++++++++++++++---------------
 4 files changed, 417 insertions(+), 260 deletions(-)

diff --git a/src/bin/system/machine.h b/src/bin/system/machine.h
index c8340c3..be8cdc6 100644
--- a/src/bin/system/machine.h
+++ b/src/bin/system/machine.h
@@ -46,6 +46,9 @@ typedef struct
 {
    char   *name;
    char   *child_name;
+#if defined(__linux__)
+   char   *path;
+#endif
    double  value;
    bool    invalid;
 } sensor_t;
@@ -137,9 +140,18 @@ system_memory_usage_get(meminfo_t *memory);
 sensor_t **
 system_sensors_thermal_get(int *count);
 
+int
+system_sensor_thermal_by_id(char *id, double *value);
+
+void
+system_sensors_thermal_free(sensor_t **sensors, int count);
+
 void
 system_power_state_get(power_t *power);
 
+void
+system_power_state_free(power_t *power);
+
 void
 system_network_transfer_get(network_t *usage);
 
diff --git a/src/bin/system/machine/sensors.bogox 
b/src/bin/system/machine/sensors.bogox
index 47638d0..4656ec3 100644
--- a/src/bin/system/machine/sensors.bogox
+++ b/src/bin/system/machine/sensors.bogox
@@ -14,10 +14,48 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
+void
+system_sensors_thermal_free(sensor_t **sensors, int count)
+{
+   for (int i = 0; i < count; i++)
+     {
+        sensor_t *snsr = sensors[i];
+        if (snsr->name)
+          free(snsr->name);
+        if (snsr->child_name)
+          free(snsr->child_name);
+#if defined(__linux__)
+        if (snsr->path)
+          free(snsr->path);
+#endif
+        free(snsr);
+     }
+   if (sensors)
+     free(sensors);
+}
+
+int
+system_sensor_thermal_by_id(char *id, double *value)
+{
+#if defined(__linux__)
+   char *d = file_contents(id);
+   if (d)
+     {
+       *value = atof(d);
+       if (*value) *value /= 1000;
+       free(d);
+       return 1;
+     }
+   return 0;
+#endif
+   return 0;
+}
+
 sensor_t **
 system_sensors_thermal_get(int *sensor_count)
 {
    sensor_t **sensors = NULL;
+   *sensor_count = 0;
 #if defined(__OpenBSD__)
    sensor_t *sensor;
    int mibs[5] = { CTL_HW, HW_SENSORS, 0, 0, 0 };
@@ -143,7 +181,7 @@ system_sensors_thermal_get(int *sensor_count)
                   snprintf(buf, sizeof(buf), "%s/temp%d_label", link, id);
                   sensor->child_name = file_contents(buf);
                   snprintf(buf, sizeof(buf), "%s/temp%d_input", link, id);
-
+                  sensor->path = strdup(buf);
                   char *d = file_contents(buf);
                   if (d)
                     {
@@ -328,6 +366,7 @@ _battery_state_get(power_t *power)
 
 #elif defined(__linux__)
    const char *path;
+   char *link;
    struct dirent *dh;
    struct stat st;
    DIR *dir;
@@ -338,11 +377,15 @@ _battery_state_get(power_t *power)
 
         path = strsli_printf("/sys/class/power_supply/%s", 
power->batteries[i]->name);
 
-        if (((stat(path, &st) < 0)) || (S_ISLNK(st.st_mode)) || 
(!S_ISDIR(st.st_mode)))
+        if ((stat(path, &st) < 0) || (!S_ISDIR(st.st_mode)))
           continue;
 
+        link = realpath(path, NULL);
+        if (!link) return;
+
         dir = opendir(path);
-        if (!dir) return;
+        if (!dir)
+          goto done;
 
         while ((dh = readdir(dir)) != NULL)
           {
@@ -360,13 +403,13 @@ _battery_state_get(power_t *power)
         if (!naming)
           continue;
 
-        buf = 
file_contents(strsli_printf("/sys/class/power_supply/%s/%s_full", 
power->batteries[i]->name, naming));
+        buf = file_contents(strsli_printf("%s/%s_full", link, naming));
         if (buf)
           {
              power->batteries[i]->charge_full = atol(buf);
              free(buf);
           }
-        buf = file_contents(strsli_printf("/sys/class/power_supply/%s/%s_now", 
power->batteries[i]->name, naming));
+        buf = file_contents(strsli_printf("%s/%s_now", link, naming));
         if (buf)
           {
              power->batteries[i]->charge_current = atol(buf);
@@ -374,6 +417,8 @@ _battery_state_get(power_t *power)
           }
         free(naming);
      }
+done:
+   if (link) free(link);
 #endif
 }
 
@@ -381,6 +426,7 @@ void
 system_power_state_get(power_t *power)
 {
    int i;
+   memset(power, 0, sizeof(power_t));
 #if defined(__OpenBSD__)
    struct sensor snsr;
    size_t slen = sizeof(struct sensor);
@@ -422,3 +468,20 @@ system_power_state_get(power_t *power)
      }
 }
 
+void
+system_power_state_free(power_t *power)
+{
+   for (int i = 0; i < power->battery_count; i++)
+     {
+
+        if (power->batteries[i]->name)
+          free(power->batteries[i]->name);
+#if defined(__OpenBSD__)
+        if (power->batteries[i]->mibs)
+          free(power->batteries[i]->mibs);
+#endif
+        free(power->batteries[i]);
+     }
+   if (power->batteries)
+     free(power->batteries);
+}
diff --git a/src/bin/ui/ui.h b/src/bin/ui/ui.h
index 8afbda8..90b2775 100644
--- a/src/bin/ui/ui.h
+++ b/src/bin/ui/ui.h
@@ -59,11 +59,11 @@ typedef struct Ui
 
    struct
    {
-      Evas_Object *win;
-      int          width;
-      int          height;
-      Evas_Object *box;
-      Evas_Object *timer;
+      Evas_Object  *win;
+      int           width;
+      int           height;
+      Evas_Object  *box;
+      Ecore_Thread *thread;
    } sensors;
 
    struct
diff --git a/src/bin/ui/ui_sensors.c b/src/bin/ui/ui_sensors.c
index 9db6635..3124364 100644
--- a/src/bin/ui/ui_sensors.c
+++ b/src/bin/ui/ui_sensors.c
@@ -1,261 +1,260 @@
 #include "ui_sensors.h"
 #include "system/machine.h"
 
-static Eina_Bool
-_battery_usage_add(Evas_Object *box, power_t *power)
+static Eina_Lock _lock;
+
+typedef struct
 {
-   Evas_Object *frame, *vbox, *hbox, *pb, *ic, *label;
-   const char *fmt;
+   Eina_List              *sensors;
+   Eina_List              *batteries;
 
-   for (int i = 0; i < power->battery_count; i++)
-     {
-        if (!power->batteries[i]->present)
-          continue;
-
-        frame = elm_frame_add(box);
-        evas_object_size_hint_align_set(frame, FILL, FILL);
-        evas_object_size_hint_weight_set(frame, EXPAND, EXPAND);
-        elm_object_style_set(frame, "pad_small");
-        evas_object_show(frame);
-
-        vbox = elm_box_add(box);
-        evas_object_size_hint_align_set(vbox, FILL, FILL);
-        evas_object_size_hint_weight_set(vbox, EXPAND, EXPAND);
-        evas_object_show(vbox);
-
-        label = elm_label_add(box);
-        evas_object_size_hint_align_set(label, 1.0, FILL);
-        evas_object_size_hint_weight_set(label, EXPAND, EXPAND);
-        evas_object_show(label);
-        elm_box_pack_end(vbox, label);
-
-        if (power->have_ac && i == 0)
-          fmt = _("<color=#fff>%s (plugged in)</>");
-        else
-          fmt = _("<color=#fff>%s</>");
-
-        elm_object_text_set(label, eina_slstr_printf(fmt,
-                        power->batteries[i]->name));
-        hbox = elm_box_add(box);
-        evas_object_size_hint_align_set(hbox, FILL, FILL);
-        evas_object_size_hint_weight_set(hbox, EXPAND, EXPAND);
-        elm_box_horizontal_set(hbox, EINA_TRUE);
-        evas_object_show(hbox);
-
-        ic = elm_image_add(box);
-        elm_image_file_set(ic, evisum_icon_path_get("battery"), NULL);
-        evas_object_size_hint_min_set(ic, 32 * elm_config_scale_get(),
-                        32 * elm_config_scale_get());
-        evas_object_show(ic);
-        elm_box_pack_end(hbox, ic);
+   Evas_Object            *combobox;
+   Elm_Genlist_Item_Class *itc;
+   Elm_Object_Item        *selected_it;
 
-        pb = elm_progressbar_add(frame);
-        evas_object_size_hint_align_set(pb, FILL, FILL);
-        evas_object_size_hint_weight_set(pb, EXPAND, EXPAND);
-        elm_progressbar_span_size_set(pb, 1.0);
-        elm_progressbar_unit_format_set(pb, "%1.0f%%");
-        elm_progressbar_value_set(pb,
-                        (double) power->batteries[i]->percent / 100);
-        evas_object_show(pb);
+   sensor_t               *sensor;
 
-        elm_box_pack_end(hbox, pb);
-        elm_box_pack_end(vbox, hbox);
-        elm_object_content_set(frame, vbox);
-        elm_box_pack_end(box, frame);
-     }
+   Evas_Object            *power_fr;
+   Evas_Object            *thermal_pb;
 
-   return !!power->battery_count;
-}
+   Ui                     *ui;
+} Ui_Data;
+
+typedef struct {
+   Evas_Object *pb;
+} Bat;
 
-static Eina_Bool
-_sensor_usage_add(Evas_Object *box, sensor_t **sensors, int count)
+typedef struct
 {
-   Evas_Object *frame, *vbox, *hbox, *pb, *ic, *label;
-   Eina_Strbuf *name;
-   sensor_t *snsr;
+   power_t   power;
 
-   for (int i = 0; i < count; i++)
-     {
-        snsr = sensors[i];
-
-        frame = elm_frame_add(box);
-        evas_object_size_hint_align_set(frame, FILL, FILL);
-        evas_object_size_hint_weight_set(frame, EXPAND, EXPAND);
-        elm_object_style_set(frame, "pad_small");
-        evas_object_show(frame);
-
-        vbox = elm_box_add(box);
-        evas_object_size_hint_align_set(vbox, FILL, FILL);
-        evas_object_size_hint_weight_set(vbox, EXPAND, EXPAND);
-        evas_object_show(vbox);
-
-        label = elm_label_add(box);
-        evas_object_size_hint_align_set(label, 1.0, FILL);
-        evas_object_size_hint_weight_set(label, EXPAND, EXPAND);
-        evas_object_show(label);
-        elm_box_pack_end(vbox, label);
-
-        name = eina_strbuf_new();
-        eina_strbuf_append(name, snsr->name);
-        if (snsr->child_name)
-          eina_strbuf_append_printf(name, " (%s)", snsr->child_name);
-
-        elm_object_text_set(label, eina_slstr_printf("<color=#fff>%s</>",
-                        eina_strbuf_string_get(name)));
-        eina_strbuf_free(name);
-
-        hbox = elm_box_add(box);
-        evas_object_size_hint_align_set(hbox, FILL, FILL);
-        evas_object_size_hint_weight_set(hbox, EXPAND, EXPAND);
-        elm_box_horizontal_set(hbox, EINA_TRUE);
-        evas_object_show(hbox);
-
-        ic = elm_image_add(box);
-        elm_image_file_set(ic, evisum_icon_path_get("sensor"), NULL);
-        evas_object_size_hint_min_set(ic, 32 * elm_config_scale_get(),
-                        32 * elm_config_scale_get());
-        evas_object_show(ic);
-        elm_box_pack_end(hbox, ic);
+   double    thermal_temp;
+   Eina_Bool thermal_valid;
+} Data;
 
-        pb = elm_progressbar_add(frame);
-        evas_object_size_hint_align_set(pb, FILL, FILL);
-        evas_object_size_hint_weight_set(pb, EXPAND, EXPAND);
-        elm_progressbar_span_size_set(pb, 1.0);
-        elm_progressbar_unit_format_set(pb, "%1.1f°C");
-        elm_progressbar_value_set(pb,
-                        (double) snsr->value / 100);
-        evas_object_show(pb);
+static void
+_name_set(char *buf, size_t len, sensor_t *s)
+{
+   if (!s->child_name)
+     snprintf(buf, len, "%s", s->name);
+   else
+     snprintf(buf, len, "%s (%s)", s->name, s->child_name);
+}
 
-        elm_box_pack_end(hbox, pb);
-        elm_box_pack_end(vbox, hbox);
-        elm_object_content_set(frame, vbox);
-        elm_box_pack_end(box, frame);
-     }
+static int
+_sort_cb(const void *p1, const void *p2)
+{
+   sensor_t *s1, *s2;
+   char buf[64], buf2[64];
+
+   s1 = (sensor_t *) p1; s2 = (sensor_t *) p2;
+
+   _name_set(buf, sizeof(buf), s1);
+   _name_set(buf2, sizeof(buf2), s2);
 
-   return count;
+   return strcmp(buf, buf2);
 }
 
 static void
-_separator_add(Evas_Object *box)
+_sensors_refresh(Ui_Data *pd)
 {
-   Evas_Object *hbox, *sep;
-
-   hbox = elm_box_add(box);
-   elm_box_horizontal_set(hbox, EINA_TRUE);
-   evas_object_size_hint_weight_set(hbox, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(hbox, FILL, FILL);
-   evas_object_show(hbox);
-
-   sep = elm_separator_add(hbox);
-   evas_object_size_hint_weight_set(sep, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(sep, FILL, FILL);
-   elm_separator_horizontal_set(sep, EINA_TRUE);
-   evas_object_show(sep);
-
-   elm_box_pack_end(hbox, sep);
-   elm_box_pack_end(box, hbox);
+   sensor_t **ss;
+   int n;
+
+   ss = system_sensors_thermal_get(&n);
+
+   for (int i = 0; i < n; i++)
+     pd->sensors = eina_list_append(pd->sensors, ss[i]);
+
+   free(ss);
+
+   pd->sensors = eina_list_sort(pd->sensors, n, _sort_cb);
+
+   elm_genlist_clear(pd->combobox);
 }
 
 static void
-_sensors_free(power_t *power, sensor_t **sensors, int sensor_count)
+_sensors_update(void *data, Ecore_Thread *thread)
 {
-   for (int i = 0; i < power->battery_count; i++)
+   Ui_Data *pd = data;
+
+   Data *msg = malloc(sizeof(Data));
+   if (!msg) return;
+
+   while (!ecore_thread_check(thread))
      {
-        if (power->batteries[i]->name)
-          free(power->batteries[i]->name);
-#if defined(__OpenBSD__)
-        if (power->batteries[i]->mibs)
-          free(power->batteries[i]->mibs);
-#endif
-        free(power->batteries[i]);
+        if (pd->selected_it)
+          {
+             if (system_sensor_thermal_by_id(pd->sensor->path, 
&msg->thermal_temp))
+               msg->thermal_valid = 1;
+          }
+        system_power_state_get(&msg->power);
+
+        ecore_thread_feedback(thread, msg);
+        if (ecore_thread_check(thread)) break;
+
+        usleep(1000000);
      }
-   if (power->batteries)
-     free(power->batteries);
 
-   for (int i = 0; i < sensor_count; i++)
+   free(msg);
+}
+
+static void
+_sensors_update_feedback_cb(void *data, Ecore_Thread *thread, void *msgdata)
+{
+   sensor_t *s;
+   Ui_Data *pd = data;
+   Eina_List *l;
+   Data *msg = msgdata;
+
+   if (!eina_lock_take_try(&_lock)) return;
+
+   EINA_LIST_FREE(pd->sensors, s)
+     elm_genlist_item_append(pd->combobox, pd->itc, s,
+                             NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
+
+   l = eina_list_nth_list(pd->batteries, 0);
+   if (l)
      {
-        sensor_t *snsr = sensors[i];
-        if (snsr->name)
-          free(snsr->name);
-        if (snsr->child_name)
-          free(snsr->child_name);
-        free(snsr);
+        if (msg->power.have_ac)
+          elm_object_text_set(pd->power_fr, _("Power (AC)"));
+       else
+          elm_object_text_set(pd->power_fr, _("Power"));
      }
-   if (sensors)
-     free(sensors);
+   for (int i = 0; i < msg->power.battery_count; i++)
+     {
+        if (msg->power.batteries[i]->present)
+          {
+              Bat *bat = eina_list_data_get(l);
+              double perc = msg->power.batteries[i]->percent / 100;
+              elm_progressbar_value_set(bat->pb, perc);
+          }
+        l = eina_list_next(l);
+     }
+   if (msg->thermal_valid)
+     elm_progressbar_value_set(pd->thermal_pb, msg->thermal_temp / 100);
+
+   system_power_state_free(&msg->power);
+
+   eina_lock_release(&_lock);
 }
 
-static Eina_Bool
-_sensors_update(void *data)
+static void
+_item_del(void *data, Evas_Object *obj)
 {
-   Ui *ui;
-   power_t power;
-   sensor_t **sensors;
-   int sensor_count = 0;
-   Evas_Object *box, *frame;
+   sensor_t *s = data;
+   if (s->name)
+     free(s->name);
+   if (s->child_name)
+     free(s->child_name);
+#if defined(__linux__)
+   if (s->path)
+     free(s->path);
+#endif
+   free(s);
+}
 
-   ui = data;
+static void
+_combo_expanded_cb(void *data, Evas_Object *obj EINA_UNUSED, void *event_info)
+{
+   Elm_Object_Item *it;
+   Ui_Data *pd = data;
 
-   elm_box_clear(ui->sensors.box);
+   if (pd->selected_it)
+     it = pd->selected_it;
+   else
+     it = elm_genlist_selected_item_get(obj);
 
-   box = elm_box_add(ui->sensors.box);
-   evas_object_size_hint_align_set(box, FILL, FILL);
-   evas_object_size_hint_weight_set(box, EXPAND, EXPAND);
-   evas_object_show(box);
+   if (it)
+     elm_genlist_item_selected_set(it, 1);
+}
 
-   memset(&power, 0, sizeof(power));
-   system_power_state_get(&power);
+static void
+_combo_item_pressed_cb(void *data, Evas_Object *obj EINA_UNUSED, void 
*event_info)
+{
+   Ui_Data *pd;
+   Elm_Object_Item *it;
+   sensor_t *s;
+   char buf[64];
 
-   if (_battery_usage_add(box, &power))
-     _separator_add(box);
+   pd = data;
+   it = pd->selected_it = event_info;
 
-   sensors = system_sensors_thermal_get(&sensor_count);
-   if (sensors)
-     {
-        _sensor_usage_add(box, sensors, sensor_count);
-        _separator_add(box);
-     }
+   pd->sensor = s = elm_object_item_data_get(it);
+   _name_set(buf, sizeof(buf), s);
+   elm_object_text_set(obj, buf);
+
+   elm_combobox_hover_end(obj);
+}
+
+static Evas_Object *
+_content_get(void *data, Evas_Object *obj EINA_UNUSED, const char *part 
EINA_UNUSED)
+{
+   Evas_Object *bx, *lb;
+   sensor_t *s;
+   char buf[64];
+
+   if (strcmp(part, "elm.swallow.content")) return NULL;
 
-   _sensors_free(&power, sensors, sensor_count);
+   s = data;
 
-   frame = elm_frame_add(ui->sensors.box);
-   evas_object_size_hint_align_set(frame, FILL, FILL);
-   evas_object_size_hint_weight_set(frame, EXPAND, EXPAND);
-   elm_object_style_set(frame, "pad_huge");
-   evas_object_show(frame);
-   elm_object_content_set(frame, box);
-   elm_box_pack_end(ui->sensors.box, frame);
+   bx = elm_box_add(obj);
+   evas_object_size_hint_weight_set(bx, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(bx, FILL, FILL);
+   evas_object_show(bx);
 
-   return EINA_TRUE;
+   lb = elm_label_add(obj);
+   evas_object_size_hint_weight_set(lb, FILL, FILL);
+   evas_object_size_hint_align_set(lb, 0.0, EXPAND);
+   _name_set(buf, sizeof(buf), s);
+   elm_object_text_set(lb, buf);
+   evas_object_show(lb);
+
+   elm_box_pack_end(bx, lb);
+
+   return bx;
 }
 
 static void
 _win_del_cb(void *data, Evas *e EINA_UNUSED, Evas_Object *obj EINA_UNUSED,
             void *event_info EINA_UNUSED)
 {
-   Ui *ui = data;
+   Bat *bat;
+   Ui_Data *pd = data;
+   Ui *ui = pd->ui;
 
-   if (ui->sensors.timer)
-     ecore_timer_del(ui->sensors.timer);
-   ui->sensors.timer = NULL;
+   eina_lock_take(&_lock);
+   ecore_thread_cancel(ui->sensors.thread);
+   ecore_thread_wait(ui->sensors.thread, 0.5);
+   ui->sensors.thread = NULL;
 
    evas_object_del(obj);
    ui->sensors.win = NULL;
+
+   EINA_LIST_FREE(pd->batteries, bat)
+     free(bat);
+
+   elm_genlist_item_class_free(pd->itc);
+   free(pd);
+   eina_lock_release(&_lock);
+   eina_lock_free(&_lock);
 }
 
 static void
 _win_resize_cb(void *data, Evas *e, Evas_Object *obj, void *event_info)
 {
-   Ui *ui = data;
+   Ui_Data *pd = data;
 
-   evisum_ui_config_save(ui);
+   evisum_ui_config_save(pd->ui);
 }
 
 void
 ui_win_sensors_add(Ui *ui)
 {
-   Evas_Object *win, *box, *hbox, *frame, *scroller;
-   Evas_Object *table, *border, *rect;
+   Evas_Object *win, *content, *bx, *tbl, *fr;
+   Evas_Object *combo, *rec, *lb, *ic, *pb;
+   Elm_Genlist_Item_Class *itc;
+   power_t power;
    Evas_Coord x = 0, y = 0;
 
    if (ui->sensors.win)
@@ -264,6 +263,12 @@ ui_win_sensors_add(Ui *ui)
         return;
      }
 
+   Ui_Data *pd = calloc(1, sizeof(Ui_Data));
+   if (!pd) return;
+   pd->ui = ui;
+
+   eina_lock_new(&_lock);
+
    ui->sensors.win = win = elm_win_util_standard_add("evisum", _("Sensors"));
    elm_win_autodel_set(win, EINA_TRUE);
    evas_object_size_hint_weight_set(win, EXPAND, EXPAND);
@@ -271,56 +276,131 @@ ui_win_sensors_add(Ui *ui)
    evisum_ui_background_random_add(win,
                                    evisum_ui_backgrounds_enabled_get());
    evas_object_event_callback_add(win, EVAS_CALLBACK_RESIZE,
-                                  _win_resize_cb, ui);
-
-   box = elm_box_add(win);
-   evas_object_size_hint_weight_set(box, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(box, FILL, FILL);
-   evas_object_hide(box);
-
-   ui->sensors.box = hbox = elm_box_add(box);
-   evas_object_size_hint_weight_set(hbox, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(hbox, FILL, FILL);
-   evas_object_show(hbox);
-
-   frame = elm_frame_add(box);
-   elm_object_style_set(frame, "pad_small");
-   evas_object_size_hint_weight_set(frame, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(frame, FILL, FILL);
-   evas_object_show(frame);
-
-   scroller = elm_scroller_add(win);
-   evas_object_size_hint_weight_set(scroller, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(scroller, FILL, FILL);
-   elm_scroller_policy_set(scroller,
-                           ELM_SCROLLER_POLICY_OFF, ELM_SCROLLER_POLICY_AUTO);
-   evas_object_show(scroller);
-
-   border = elm_frame_add(box);
-   elm_object_style_set(border, "pad_small");
-   evas_object_size_hint_weight_set(border, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(border, FILL, FILL);
-   evas_object_show(border);
-   elm_object_content_set(border, hbox);
-
-   table = elm_table_add(win);
-   evas_object_size_hint_weight_set(table, EXPAND, EXPAND);
-   evas_object_size_hint_align_set(table, FILL, FILL);
-   evas_object_show(table);
-
-   rect = evas_object_rectangle_add(evas_object_rectangle_add(win));
-   evas_object_size_hint_max_set(rect, MISC_MAX_WIDTH, -1);
-   evas_object_size_hint_min_set(rect, MISC_MIN_WIDTH, 1);
-
-   elm_table_pack(table, rect, 0, 0, 1, 1);
-   elm_table_pack(table, border, 0, 0, 1, 1);
-
-   elm_object_content_set(scroller, table);
-   elm_object_content_set(frame,scroller);
-   elm_box_pack_end(box, frame);
-   elm_object_content_set(win, box);
-
-   evas_object_event_callback_add(win, EVAS_CALLBACK_DEL, _win_del_cb, ui);
+                                  _win_resize_cb, pd);
+   evas_object_event_callback_add(win, EVAS_CALLBACK_DEL, _win_del_cb, pd);
+
+   content = elm_box_add(win);
+   evas_object_size_hint_weight_set(content, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(content, FILL, FILL);
+   evas_object_show(content);
+   elm_object_content_set(win, content);
+
+   pd->power_fr = fr = elm_frame_add(win);
+   evas_object_size_hint_weight_set(fr, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(fr, FILL, FILL);
+   elm_object_text_set(fr, _("Power"));
+   evas_object_show(fr);
+
+   bx = elm_box_add(win);
+   evas_object_size_hint_weight_set(bx, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(bx, FILL, FILL);
+   evas_object_show(bx);
+   elm_object_content_set(fr, bx);
+
+   system_power_state_get(&power);
+
+   for (int i = 0; i < power.battery_count; i++)
+     {
+        if (!power.batteries[i]->present) continue;
+
+        Bat *bat = calloc(1, sizeof(Bat));
+        if (!bat) return;
+
+        tbl = elm_table_add(win);
+        evas_object_size_hint_weight_set(tbl, EXPAND, 0.5);
+        evas_object_size_hint_align_set(tbl, FILL, FILL);
+        evas_object_show(tbl);
+        elm_box_pack_end(bx, tbl);
+
+        rec = evas_object_rectangle_add(evas_object_evas_get(win));
+        evas_object_size_hint_min_set(rec, 1, ELM_SCALE_SIZE(48));
+        elm_table_pack(tbl, rec, 0, 0, 1, 1);
+
+        ic = elm_icon_add(win);
+        elm_icon_standard_set(ic, evisum_icon_path_get("battery"));
+        evas_object_size_hint_min_set(ic, ELM_SCALE_SIZE(32), 
ELM_SCALE_SIZE(32));
+        evas_object_show(ic);
+        elm_table_pack(tbl, ic, 0, 0, 1, 1);
+        elm_object_content_set(fr, bx);
+        elm_box_pack_end(content, fr);
+
+        lb = elm_label_add(win);
+        elm_object_text_set(lb, eina_slstr_printf("<small>%s</>", 
power.batteries[i]->name));
+        evas_object_size_hint_weight_set(lb, EXPAND, 0);
+        evas_object_size_hint_align_set(lb, 0.5, 0.5);
+        evas_object_show(lb);
+
+        pb = elm_progressbar_add(win);
+        evas_object_size_hint_weight_set(pb, EXPAND, EXPAND);
+        evas_object_size_hint_align_set(pb, FILL, FILL);
+        evas_object_show(pb);
+        bat->pb = pb;
+
+        elm_table_pack(tbl, pb, 1, 0, 1, 1);
+        elm_table_pack(tbl, lb, 1, 0, 1, 1);
+
+        elm_box_pack_end(bx, tbl);
+
+        pd->batteries = eina_list_append(pd->batteries, bat);
+     }
+
+   system_power_state_free(&power);
+
+   elm_box_pack_end(content, fr);
+
+   fr = elm_frame_add(win);
+   evas_object_size_hint_weight_set(fr, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(fr, FILL, FILL);
+   elm_object_text_set(fr, _("Thermal"));
+   evas_object_show(fr);
+
+   bx = elm_box_add(win);
+   evas_object_size_hint_weight_set(bx, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(bx, FILL, FILL);
+   evas_object_show(bx);
+
+   pd->combobox = combo = elm_combobox_add(win);
+   evas_object_size_hint_weight_set(combo, EXPAND, 0);
+   evas_object_size_hint_align_set(combo, FILL, FILL);
+   elm_object_text_set(combo, _("Select..."));
+   elm_genlist_multi_select_set(combo, 0);
+   evas_object_smart_callback_add(combo, "item,pressed", 
_combo_item_pressed_cb, pd);
+   evas_object_smart_callback_add(combo, "expanded", _combo_expanded_cb, pd);
+   evas_object_show(combo);
+   elm_box_pack_end(bx, combo);
+
+   tbl = elm_table_add(win);
+   evas_object_size_hint_weight_set(tbl, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(tbl, FILL, FILL);
+   elm_table_align_set(tbl, 0, 0.5);
+   evas_object_show(tbl);
+   elm_box_pack_end(bx, tbl);
+
+   rec = evas_object_rectangle_add(evas_object_evas_get(win));
+   evas_object_size_hint_min_set(rec, 1, ELM_SCALE_SIZE(48));
+   elm_table_pack(tbl, rec, 0, 0, 1, 1);
+
+   ic = elm_icon_add(win);
+   elm_icon_standard_set(ic, evisum_icon_path_get("sensor"));
+   evas_object_size_hint_min_set(ic, ELM_SCALE_SIZE(32), ELM_SCALE_SIZE(32));
+   evas_object_show(ic);
+   elm_table_pack(tbl, ic, 0, 0, 1, 1);
+   elm_object_content_set(fr, bx);
+   elm_box_pack_end(content, fr);
+
+   pd->thermal_pb = pb = elm_progressbar_add(win);
+   evas_object_size_hint_weight_set(pb, EXPAND, EXPAND);
+   evas_object_size_hint_align_set(pb, FILL, FILL);
+   elm_progressbar_unit_format_set(pb, "%1.1f°C");
+   evas_object_show(pb);
+   elm_table_pack(tbl, pb, 1, 0, 1, 1);
+
+   pd->itc = itc = elm_genlist_item_class_new();
+   itc->item_style = "full";
+   itc->func.content_get = _content_get;
+   itc->func.text_get = NULL;
+   itc->func.filter_get = NULL;
+   itc->func.del = _item_del;
 
    if (ui->sensors.width > 0 && ui->sensors.height > 0)
      evas_object_resize(win, ui->sensors.width, ui->sensors.height);
@@ -336,8 +416,10 @@ ui_win_sensors_add(Ui *ui)
 
    evas_object_show(win);
 
-   _sensors_update(ui);
+   _sensors_refresh(pd);
 
-   ui->sensors.timer = ecore_timer_add(ui->settings.poll_delay, 
_sensors_update, ui);
+   ui->sensors.thread = ecore_thread_feedback_run(_sensors_update,
+                                                  _sensors_update_feedback_cb,
+                                                  NULL, NULL, pd, EINA_TRUE);
 }
 

-- 


Reply via email to