From: Carsten Haitzler (Rasterman) <raster@rasterman.com>
Date: Thu, 31 Dec 2015 06:42:25 +0000 (+0900)
Subject: elm config tool - dont need to call save if flush is saving anyway
X-Git-Tag: upstream/1.20.0~7116^2~14^2~317
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=76c528a5c304c464d9d6c67b0b36937192c82bdf;p=platform%2Fupstream%2Fefl.git

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.
---

diff --git a/legacy/elementary/src/bin/config.c b/legacy/elementary/src/bin/config.c
index 6733be6..66924cf 100644
--- a/legacy/elementary/src/bin/config.c
+++ b/legacy/elementary/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();
      }
 }