raster pushed a commit to branch master.

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

commit f0ea0c94cadb0b8100ad10603f52f35208e3fe97
Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
Date:   Tue Sep 22 14:06:40 2015 +0900

    e config - cache sizes in config don't actually work, so remove
    
    remove several config vars in advanced performance that frankly don't
    work (unless you change them int he dialog). they are not set up on
    startup or not even used at all. remove things that don't work anymore!
    
    @fix
---
 src/bin/e_config.c                                 | 10 ---
 src/bin/e_config.h                                 |  7 +-
 src/bin/e_module.h                                 |  2 +-
 .../conf_performance/e_int_config_performance.c    | 82 +++-------------------
 4 files changed, 10 insertions(+), 91 deletions(-)

diff --git a/src/bin/e_config.c b/src/bin/e_config.c
index fbaefd6..0d5a22d 100644
--- a/src/bin/e_config.c
+++ b/src/bin/e_config.c
@@ -438,10 +438,6 @@ _e_config_edd_init(Eina_Bool old)
    E_CONFIG_VAL(D, T, border_shade_speed, DOUBLE); /**/
    E_CONFIG_VAL(D, T, framerate, DOUBLE); /**/
    E_CONFIG_VAL(D, T, priority, INT); /**/
-   E_CONFIG_VAL(D, T, image_cache, INT); /**/
-   E_CONFIG_VAL(D, T, font_cache, INT); /**/
-   E_CONFIG_VAL(D, T, edje_cache, INT); /**/
-   E_CONFIG_VAL(D, T, edje_collection_cache, INT); /**/
    E_CONFIG_VAL(D, T, zone_desks_x_count, INT); /**/
    E_CONFIG_VAL(D, T, zone_desks_y_count, INT); /**/
    E_CONFIG_VAL(D, T, show_desktop_icons, INT); /**/
@@ -646,7 +642,6 @@ _e_config_edd_init(Eina_Bool old)
    E_CONFIG_VAL(D, T, menu_gadcon_client_toplevel, INT);
 
    E_CONFIG_VAL(D, T, ping_clients_interval, INT);
-   E_CONFIG_VAL(D, T, cache_flush_poll_interval, INT);
 
    E_CONFIG_VAL(D, T, thumbscroll_enable, INT);
    E_CONFIG_VAL(D, T, thumbscroll_threshhold, INT);
@@ -1348,11 +1343,6 @@ e_config_load(void)
    E_CONFIG_LIMIT(e_config->border_shade_speed, 1.0, 20000.0);
    E_CONFIG_LIMIT(e_config->framerate, 1.0, 200.0);
    E_CONFIG_LIMIT(e_config->priority, 0, 19);
-   E_CONFIG_LIMIT(e_config->image_cache, 0, 256 * 1024);
-   E_CONFIG_LIMIT(e_config->font_cache, 0, 32 * 1024);
-   E_CONFIG_LIMIT(e_config->edje_cache, 0, 256);
-   E_CONFIG_LIMIT(e_config->edje_collection_cache, 0, 512);
-   E_CONFIG_LIMIT(e_config->cache_flush_poll_interval, 8, 32768);
    E_CONFIG_LIMIT(e_config->zone_desks_x_count, 1, 64);
    E_CONFIG_LIMIT(e_config->zone_desks_y_count, 1, 64);
    E_CONFIG_LIMIT(e_config->show_desktop_icons, 0, 1);
diff --git a/src/bin/e_config.h b/src/bin/e_config.h
index c3362f2..7fe67c4 100644
--- a/src/bin/e_config.h
+++ b/src/bin/e_config.h
@@ -71,10 +71,6 @@ struct _E_Config
    double      border_shade_speed; // GUI
    double      framerate; // GUI
    int         priority; // GUI
-   int         image_cache; // GUI
-   int         font_cache; // GUI
-   int         edje_cache; // GUI
-   int         edje_collection_cache; // GUI
    int         zone_desks_x_count; // GUI
    int         zone_desks_y_count; // GUI
    int         show_desktop_icons; // GUI
@@ -282,13 +278,12 @@ struct _E_Config
 
    Eina_List                *mime_icons; // GUI
    int                       desk_auto_switch; // GUI;
-   
+
    int                       screen_limits;
 
    int                       thumb_nice;
 
    int                       ping_clients_interval; // GUI
-   int                       cache_flush_poll_interval; // GUI
 
    int                       thumbscroll_enable; // GUI
    int                       thumbscroll_threshhold; // GUI
diff --git a/src/bin/e_module.h b/src/bin/e_module.h
index 07a51f8..192810e 100644
--- a/src/bin/e_module.h
+++ b/src/bin/e_module.h
@@ -1,6 +1,6 @@
 #ifdef E_TYPEDEFS
 
-#define E_MODULE_API_VERSION 16
+#define E_MODULE_API_VERSION 17
 
 typedef struct _E_Module     E_Module;
 typedef struct _E_Module_Api E_Module_Api;
diff --git a/src/modules/conf_performance/e_int_config_performance.c 
b/src/modules/conf_performance/e_int_config_performance.c
index f99c84b..d420ecf 100644
--- a/src/modules/conf_performance/e_int_config_performance.c
+++ b/src/modules/conf_performance/e_int_config_performance.c
@@ -11,11 +11,6 @@ struct _E_Config_Dialog_Data
    double framerate;
    int priority;
    int module_delay;
-   int cache_flush_poll_interval;
-   double font_cache;
-   double image_cache;
-   int edje_cache;
-   int edje_collection_cache;
 };
 
 E_Config_Dialog *
@@ -34,8 +29,8 @@ e_int_config_performance(Evas_Object *parent EINA_UNUSED, 
const char *params EIN
    v->basic.check_changed = _basic_check_changed;
 
    cfd = e_config_dialog_new(NULL, _("Performance Settings"),
-                            "E", "advanced/performance",
-                            "preferences-system-performance", 0, v, NULL);
+                             "E", "advanced/performance",
+                             "preferences-system-performance", 0, v, NULL);
    return cfd;
 }
 
@@ -49,11 +44,6 @@ _create_data(E_Config_Dialog *cfd EINA_UNUSED)
    cfdata->framerate = e_config->framerate;
    cfdata->priority = e_config->priority;
    cfdata->module_delay = !e_config->no_module_delay;
-   cfdata->font_cache = ((double)e_config->font_cache / 1024);
-   cfdata->image_cache = ((double)e_config->image_cache / 1024);
-   cfdata->edje_cache = e_config->edje_cache;
-   cfdata->edje_collection_cache = e_config->edje_collection_cache;
-   cfdata->cache_flush_poll_interval = e_config->cache_flush_poll_interval;
    return cfdata;
 }
 
@@ -68,17 +58,10 @@ _basic_apply(E_Config_Dialog *cfd EINA_UNUSED, 
E_Config_Dialog_Data *cfdata)
 {
    if (cfdata->framerate <= 0.0) cfdata->framerate = 1.0;
    e_config->framerate = cfdata->framerate;
-   e_config->cache_flush_poll_interval = cfdata->cache_flush_poll_interval;
-   e_config->font_cache = (cfdata->font_cache * 1024);
-   e_config->image_cache = (cfdata->image_cache * 1024);
-   e_config->edje_cache = cfdata->edje_cache;
-   e_config->edje_collection_cache = cfdata->edje_collection_cache;
    edje_frametime_set(1.0 / e_config->framerate);
    e_config->priority = cfdata->priority;
    e_config->no_module_delay = !cfdata->module_delay;
    ecore_exe_run_priority_set(e_config->priority);
-   elm_config_cache_image_cache_size_set(cfdata->image_cache);
-   elm_config_cache_font_cache_size_set(cfdata->font_cache);
    e_config_save_queue();
    return 1;
 }
@@ -88,79 +71,30 @@ _basic_check_changed(E_Config_Dialog *cfd EINA_UNUSED, 
E_Config_Dialog_Data *cfd
 {
    if (cfdata->framerate <= 0.0) cfdata->framerate = 1.0;
    return ((e_config->framerate != cfdata->framerate) ||
-          (e_config->cache_flush_poll_interval != 
cfdata->cache_flush_poll_interval) ||
-          (e_config->font_cache != (cfdata->font_cache * 1024)) ||
-          (e_config->image_cache != (cfdata->image_cache * 1024)) ||
-          (e_config->edje_cache != cfdata->edje_cache) ||
-          (e_config->edje_collection_cache != cfdata->edje_collection_cache) ||
-          (e_config->priority != cfdata->priority) ||
+           (e_config->priority != cfdata->priority) ||
            (e_config->no_module_delay != (!cfdata->module_delay)));
 }
 
 static Evas_Object *
 _basic_create(E_Config_Dialog *cfd EINA_UNUSED, Evas *evas, 
E_Config_Dialog_Data *cfdata)
 {
-   Evas_Object *otb, *ob, *ol;
-
-   otb = e_widget_toolbook_add(evas, (24 * e_scale), (24 * e_scale));
+   Evas_Object *ob, *ol;
 
    ol = e_widget_list_add(evas, 0, 0);
    ob = e_widget_label_add(evas, _("Framerate"));
    e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, _("%1.0f fps"), 5.0, 200.0, 1.0, 0, 
+   ob = e_widget_slider_add(evas, 1, 0, _("%1.0f fps"), 5.0, 200.0, 1.0, 0,
                             &(cfdata->framerate), NULL, 100);
    e_widget_list_object_append(ol, ob, 1, 0, 0.5);
 
    ob = e_widget_label_add(evas, _("Application priority"));
    e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, "%1.0f", 0, 19, 1, 0, NULL, 
+   ob = e_widget_slider_add(evas, 1, 0, "%1.0f", 0, 19, 1, 0, NULL,
                             &(cfdata->priority), 100);
    e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-   
-   ob = e_widget_check_add(evas, _("Allow module load delay"), 
&(cfdata->module_delay));
-   e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-                                                               
-   e_widget_toolbook_page_append(otb, NULL, _("General"), ol, 
-                                 1, 0, 1, 0, 0.5, 0.0);
 
-   ol = e_widget_list_add(evas, 0, 0);
-   ob = e_widget_label_add(evas, _("Cache flush interval"));
-   e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, _("%1.0f ticks"), 8, 4096, 8, 0, NULL,
-                            &(cfdata->cache_flush_poll_interval), 100);
-   e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-
-   ob = e_widget_label_add(evas, _("Font cache size"));
-   e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, _("%1.1f MiB"), 0, 4, 0.1, 0,
-                            &(cfdata->font_cache), NULL, 100);
-   e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-
-   ob = e_widget_label_add(evas, _("Image cache size"));
-   e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, _("%1.0f MiB"), 0, 32, 1, 0,
-                            &(cfdata->image_cache), NULL, 100);
-   e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-
-   e_widget_toolbook_page_append(otb, NULL, _("Caches"), ol, 
-                                 1, 0, 1, 0, 0.5, 0.0);
-
-   ol = e_widget_list_add(evas, 0, 0);
-   ob = e_widget_label_add(evas, _("Number of Edje files to cache"));
-   e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, _("%1.0f files"), 0, 256, 1, 0, NULL,
-                            &(cfdata->edje_cache), 100);
-   e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-
-   ob = e_widget_label_add(evas, _("Number of Edje collections to cache"));
-   e_widget_list_object_append(ol, ob, 1, 1, 0.5);
-   ob = e_widget_slider_add(evas, 1, 0, _("%1.0f collections"), 0, 512, 1, 0,
-                            NULL, &(cfdata->edje_collection_cache), 100);
+   ob = e_widget_check_add(evas, _("Allow module load delay"), 
&(cfdata->module_delay));
    e_widget_list_object_append(ol, ob, 1, 0, 0.5);
-   e_widget_toolbook_page_append(otb, NULL, _("Edje Cache"), ol, 
-                                 1, 0, 1, 0, 0.5, 0.0);
-
-   e_widget_toolbook_page_show(otb, 0);
 
-   return otb;
+   return ol;
 }

-- 


Reply via email to