netstar pushed a commit to branch master.

http://git.enlightenment.org/apps/evisum.git/commit/?id=704694e688fd787d1ad0d4e617b9a506e5c05b29

commit 704694e688fd787d1ad0d4e617b9a506e5c05b29
Author: Alastair Poole <nets...@gmail.com>
Date:   Sat Apr 18 03:33:59 2020 +0100

    ui: improved and ready to tidy for releasing.
---
 src/bin/ui.c | 134 ++++-------------------------------------------------------
 src/bin/ui.h |  10 +----
 2 files changed, 9 insertions(+), 135 deletions(-)

diff --git a/src/bin/ui.c b/src/bin/ui.c
index 4bd90e8..e1f4f8c 100644
--- a/src/bin/ui.c
+++ b/src/bin/ui.c
@@ -415,7 +415,7 @@ _tab_memory_update(Ui *ui, results_t *results)
    if (!ui->mem_visible)
      return;
 
-   elm_object_text_set(ui->label_mem, 
eina_slstr_printf("<subtitle>Memory</subtitle><br>" \
+   elm_object_text_set(ui->title_mem, 
eina_slstr_printf("<subtitle>Memory</subtitle><br>" \
                        "<bigger>Physical %lu %c</bigger><br>" \
                        "Swap %lu %c",
                        _mem_adjust(ui->data_unit, results->memory.total), 
ui->data_unit,
@@ -822,86 +822,6 @@ _proc_pid_cpu_usage_get(Ui *ui, Proc_Info *proc)
    _proc_pid_cpu_time_save(ui, proc);
 }
 
-static void
-_column_expand_init(Ui *ui)
-{
-   ui->long_pid   = strdup("");
-   ui->long_uid   = strdup("");
-   ui->long_size  = strdup("");
-   ui->long_rss   = strdup("");
-   ui->long_cmd   = strdup("");
-   ui->long_state = strdup("");
-   ui->long_cpu_usage = strdup("");
-}
-
-static void
-_column_expand_calculate(Ui *ui, Proc_Info *proc)
-{
-   const char *text;
-
-   text = eina_slstr_printf("%d", proc->pid);
-   if (strlen(text) > strlen(ui->long_pid))
-     {
-        if (ui->long_pid) free(ui->long_pid);
-        ui->long_pid = strdup(text);
-     }
-
-   text = eina_slstr_printf("%d", proc->uid);
-   if (strlen(text) > strlen(ui->long_uid))
-     {
-       if (ui->long_uid) free(ui->long_uid);
-       ui->long_uid = strdup(text);
-     }
-
-   text = eina_slstr_printf("%lu %c ", _mem_adjust(ui->data_unit, 
proc->mem_size >> 10), ui->data_unit);
-   if (strlen(text) > strlen(ui->long_size))
-     {
-        if (ui->long_size) free(ui->long_size);
-        ui->long_size = strdup(text);
-     }
-
-   text = eina_slstr_printf("%lu %c ", _mem_adjust(ui->data_unit, 
proc->mem_rss >> 10), ui->data_unit);
-   if (strlen(text) > strlen(ui->long_rss))
-     {
-        if (ui->long_rss) free(ui->long_rss);
-        ui->long_rss = strdup(text);
-     }
-
-   text = eina_slstr_printf("%sXXX", proc->command);
-   if (strlen(text) > strlen(ui->long_cmd))
-     {
-        if (ui->long_cmd) free(ui->long_cmd);
-        ui->long_cmd = strdup(text);
-     }
-
-   text = eina_slstr_printf("%s", proc->state);
-   if (strlen(text) > strlen(ui->long_state))
-     {
-        if (ui->long_state) free(ui->long_state);
-        ui->long_state = strdup(text);
-     }
-
-   text = eina_slstr_printf("%.1f%%", proc->cpu_usage);
-   if (strlen(text) > strlen(ui->long_cpu_usage))
-     {
-        if (ui->long_cpu_usage) free(ui->long_cpu_usage);
-        ui->long_cpu_usage = strdup(text);
-     }
-}
-
-static Evas_Object *
-_label_expand(Evas_Object *parent, const char *text)
-{
-   Evas_Object *label;
-
-   label = elm_label_add(parent);
-   evas_object_data_set(label, text, label);
-   evas_object_size_hint_align_set(label, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
-   evas_object_size_hint_weight_set(label, EVAS_HINT_FILL, EVAS_HINT_FILL);
-
-   return label;
-}
-
 static void
 _item_unrealized_cb(void *data, Evas_Object *obj EINA_UNUSED, void *event_info 
EINA_UNUSED)
 {
@@ -955,7 +875,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, rect, 0, 0, 1, 1);
    elm_table_pack(table, label, 0, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_pid"), 0, 0, 1, 1);
 
    label = elm_label_add(table);
    evas_object_data_set(table, "proc_uid", label);
@@ -966,7 +885,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, rect, 1, 0, 1, 1);
    elm_table_pack(table, label, 1, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_uid"), 1, 0, 1, 1);
 
    label = elm_label_add(table);
    evas_object_data_set(table, "proc_size", label);
@@ -977,7 +895,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, rect, 2, 0, 1, 1);
    elm_table_pack(table, label, 2, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_size"), 2, 0, 1, 1);
 
    label = elm_label_add(table);
    evas_object_data_set(table, "proc_rss", label);
@@ -988,7 +905,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, rect, 3, 0, 1, 1);
    elm_table_pack(table, label, 3, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_rss"), 3, 0, 1, 1);
 
    label = elm_label_add(table);
    evas_object_data_set(table, "proc_cmd", label);
@@ -999,7 +915,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, rect, 4, 0, 1, 1);
    elm_table_pack(table, label, 4, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_cmd"), 4, 0, 1, 1);
 
    label = elm_label_add(table);
    evas_object_data_set(table, "proc_state", label);
@@ -1010,7 +925,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, label, 5, 0, 1, 1);
    elm_table_pack(table, rect, 5, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_state"), 5, 0, 1, 1);
 
    label = elm_label_add(table);
    evas_object_data_set(table, "proc_cpu_usage", label);
@@ -1021,7 +935,6 @@ _item_create(Evas_Object *parent)
    evas_object_data_set(label, "rect", rect);
    elm_table_pack(table, label, 6, 0, 1, 1);
    elm_table_pack(table, rect, 6, 0, 1, 1);
-   elm_table_pack(table, _label_expand(table, "long_cpu_usage"), 6, 0, 1, 1);
 
    return table;
 }
@@ -1084,75 +997,46 @@ _content_get(void *data, Evas_Object *obj, const char 
*source)
 
    evas_object_geometry_get(ui->btn_pid, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_pid");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%d", proc->pid));
+   elm_object_text_set(l, eina_slstr_printf("%d", proc->pid));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
    evas_object_geometry_get(ui->btn_uid, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_uid");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%d", proc->uid));
+   elm_object_text_set(l, eina_slstr_printf("%d", proc->uid));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
    evas_object_geometry_get(ui->btn_size, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_size");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%lu %c ", 
_mem_adjust(ui->data_unit, proc->mem_size >> 10), ui->data_unit));
+   elm_object_text_set(l, eina_slstr_printf("%lu %c ", 
_mem_adjust(ui->data_unit, proc->mem_size >> 10), ui->data_unit));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
    evas_object_geometry_get(ui->btn_rss, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_rss");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%lu %c ", 
_mem_adjust(ui->data_unit, proc->mem_rss >> 10), ui->data_unit));
+   elm_object_text_set(l, eina_slstr_printf("%lu %c ", 
_mem_adjust(ui->data_unit, proc->mem_rss >> 10), ui->data_unit));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
    evas_object_geometry_get(ui->btn_cmd, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_cmd");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%s", proc->command));
+   elm_object_text_set(l, eina_slstr_printf("%s", proc->command));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
    evas_object_geometry_get(ui->btn_state, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_state");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%s", proc->state));
+   elm_object_text_set(l, eina_slstr_printf("%s", proc->state));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
    evas_object_geometry_get(ui->btn_cpu_usage, NULL, NULL, &w, &h);
    l = evas_object_data_get(it->obj, "proc_cpu_usage");
-   if (l)
-     elm_object_text_set(l, eina_slstr_printf("%.1f%%", proc->cpu_usage));
+   elm_object_text_set(l, eina_slstr_printf("%.1f%%", proc->cpu_usage));
    r = evas_object_data_get(l, "rect");
    evas_object_size_hint_min_set(r, w, 1);
 
-   l = evas_object_data_get(it->obj, "long_pid");
-   if (l)
-     elm_object_text_set(l,  ui->long_pid);
-   l = evas_object_data_get(it->obj, "long_uid");
-   if (l)
-     elm_object_text_set(l,  ui->long_uid);
-   l = evas_object_data_get(it->obj, "long_size");
-   if (l)
-     elm_object_text_set(l, ui->long_size);
-   l = evas_object_data_get(it->obj, "long_rss");
-   if (l)
-     elm_object_text_set(l, ui->long_rss);
-   l = evas_object_data_get(it->obj, "long_cmd");
-   if (l)
-     elm_object_text_set(l, ui->long_cmd);
-   l = evas_object_data_get(it->obj, "long_state");
-   if (l)
-     elm_object_text_set(l, ui->long_state);
-   l = evas_object_data_get(it->obj, "long_cpu_usage");
-   if (l)
-     elm_object_text_set(l, ui->long_cpu_usage);
-
    return it->obj;
 }
 
@@ -1218,7 +1102,6 @@ _process_list_feedback_cb(void *data, Ecore_Thread 
*thread EINA_UNUSED, void *ms
         else
          {
             _proc_pid_cpu_usage_get(ui, proc);
-            _column_expand_calculate(ui, proc);
          }
      }
 
@@ -2814,7 +2697,6 @@ _ui_init(Evas_Object *parent)
    _ui_tab_misc_add(ui);
 
    _item_cache_init(ui);
-   _column_expand_init(ui);
 
    return ui;
 }
diff --git a/src/bin/ui.h b/src/bin/ui.h
index c76f78a..d6e5983 100644
--- a/src/bin/ui.h
+++ b/src/bin/ui.h
@@ -85,12 +85,12 @@ typedef struct Ui
    Evas_Object  *progress_cpu;
    Evas_Object  *progress_mem;
 
+   Evas_Object  *title_mem;
    Evas_Object  *progress_mem_used;
    Evas_Object  *progress_mem_cached;
    Evas_Object  *progress_mem_buffered;
    Evas_Object  *progress_mem_shared;
    Evas_Object  *progress_mem_swap;
-   Evas_Object  *label_mem;
 
    Evas_Object  *btn_pid;
    Evas_Object  *btn_uid;
@@ -100,14 +100,6 @@ typedef struct Ui
    Evas_Object  *btn_state;
    Evas_Object  *btn_cpu_usage;
 
-   char         *long_pid;
-   char         *long_uid;
-   char         *long_size;
-   char         *long_rss;
-   char         *long_cmd;
-   char         *long_state;
-   char         *long_cpu_usage;
-
    Evas_Object  *genlist_procs;
 
    Evas_Object  *entry_pid_cmd;

-- 


Reply via email to