[EGIT] [core/elementary] master 02/06: elm config tool - dont need to call save if flush is saving anyway

[Carsten Haitzler]

raster pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=37a708d92892aeff0e8ce75c365a58d79fab5bbc

commit 37a708d92892aeff0e8ce75c365a58d79fab5bbc
Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
Date:   Thu Dec 31 15:42:25 2015 +0900

    elm config tool - dont need to call save if flush is saving anyway
    
    tis would also make behavior more consistent with the tool as before
    sometimes it flushed + saved, sometimes just flushed which would lose
    settings. now since this works right in elm config in the lib, we dont
    need to save if we flush as flush saves too.
---
 src/bin/config.c | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/src/bin/config.c b/src/bin/config.c
index 6733be6..66924cf 100644
--- a/src/bin/config.c
+++ b/src/bin/config.c
@@ -856,7 +856,6 @@ sel_change(void *data       EINA_UNUSED,
    if (val == sel) return;
    elm_config_selection_unfocused_clear_set(val);
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -870,7 +869,6 @@ dbg_change(void *data       EINA_UNUSED,
    if (val == sel) return;
    elm_config_clouseau_enabled_set(val);
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -884,7 +882,6 @@ atspi_change(void *data       EINA_UNUSED,
    if (val == sel) return;
    elm_config_atspi_mode_set(val);
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -897,7 +894,6 @@ transition_duration_change(void *data EINA_UNUSED,
 
    if (scale == val) return;
    elm_config_transition_duration_factor_set(val);
-   elm_config_save();
    elm_config_all_flush();
 }
 
@@ -1132,7 +1128,6 @@ _font_overlay_set_all(void            *data,
      }
 
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -1170,7 +1165,6 @@ _font_overlay_reset(void            *data,
    ELM_LIST_DISABLE(fsizes);
 
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -1210,7 +1204,6 @@ _font_overlay_reset_all(void            *data,
    ELM_LIST_DISABLE(fsizes);
 
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -1239,7 +1232,6 @@ _font_overlay_change(void *data       EINA_UNUSED,
 
    elm_config_font_overlay_apply();
    elm_config_all_flush();
-   elm_config_save();
 
    /* TODO: apply hinting */
 }
@@ -1411,7 +1403,6 @@ _profile_use(void            *data,
 
    _config_display_update(elm_object_top_widget_get(li));
    elm_config_all_flush();
-   elm_config_save(); /* make sure new profile has its data dir */
 }
 
 static void
@@ -1436,7 +1427,6 @@ _profile_reset(void            *data,
    if (!selection) return;
 
    elm_config_all_flush();
-   elm_config_save(); /* dump config into old profile's data dir */
 
    pdir = elm_config_profile_dir_get(selection, EINA_TRUE);
    if (!pdir)
@@ -1449,7 +1439,6 @@ _profile_reset(void            *data,
 
    elm_config_all_flush();
    _config_display_update(elm_object_top_widget_get(li));
-   elm_config_save(); /* make sure new profile has its data dir */
 }
 
 static void
@@ -1466,7 +1455,6 @@ _theme_use(void *data       EINA_UNUSED,
    defth = elm_theme_get(th);
    elm_theme_set(NULL, defth);
    elm_config_all_flush();
-   elm_config_save();
 }
 
 static void
@@ -3580,7 +3568,6 @@ _cb_accel(void *data, Evas_Object *obj EINA_UNUSED, void 
*info EINA_UNUSED)
      {
         elm_config_accel_preference_set(val);
         elm_config_all_flush();
-        elm_config_save();
      }
 }
 
@@ -3594,7 +3581,6 @@ _cb_accel_override(void *data EINA_UNUSED, Evas_Object 
*obj, void *info EINA_UNU
      {
         elm_config_accel_preference_override_set(val);
         elm_config_all_flush();
-        elm_config_save();
      }
 }
 
@@ -3608,7 +3594,6 @@ _cb_vsync(void *data EINA_UNUSED, Evas_Object *obj, void 
*info EINA_UNUSED)
      {
         elm_config_vsync_set(val);
         elm_config_all_flush();
-        elm_config_save();
      }
 }
 

--