Enlightenment CVS committal Author : devilhorns Project : e17 Module : apps/e
Dir : e17/apps/e/src/modules/fileman Modified Files: e_mod_main.c e_mod_main.h Log Message: Implement Config File Version support for Fileman module as I know more options will get added in the future. Sorry for the initial config wipe people but this way we can avoid future problems. =================================================================== RCS file: /cvs/e/e17/apps/e/src/modules/fileman/e_mod_main.c,v retrieving revision 1.4 retrieving revision 1.5 diff -u -3 -r1.4 -r1.5 --- e_mod_main.c 13 Aug 2007 00:00:13 -0000 1.4 +++ e_mod_main.c 13 Aug 2007 05:05:26 -0000 1.5 @@ -10,6 +10,8 @@ static void _e_mod_fileman_cb(void *data, E_Menu *m, E_Menu_Item *mi); static void _e_mod_menu_add(void *data, E_Menu *m); static void _e_mod_fileman_config_load(void); +static void _e_mod_fileman_config_free(void); +static int _e_mod_cb_config_timer(void *data); static E_Module *conf_module = NULL; static E_Action *act = NULL; @@ -130,7 +132,7 @@ e_configure_registry_item_del("fileman/fileman"); e_configure_registry_category_del("fileman"); - E_FREE(fileman_config); + _e_mod_fileman_config_free(); E_CONFIG_DD_FREE(conf_edd); conf_module = NULL; return 1; @@ -223,6 +225,7 @@ #undef D #define T Config #define D conf_edd + E_CONFIG_VAL(D, T, config_version, INT); E_CONFIG_VAL(D, T, view.mode, INT); E_CONFIG_VAL(D, T, view.open_dirs_in_place, UCHAR); E_CONFIG_VAL(D, T, view.selector, UCHAR); @@ -252,31 +255,86 @@ E_CONFIG_VAL(D, T, theme.fixed, UCHAR); fileman_config = e_config_domain_load("module.fileman", conf_edd); + if (fileman_config) + { + if ((fileman_config->config_version >> 16) < MOD_CONFIG_FILE_EPOCH) + { + _e_mod_fileman_config_free(); + ecore_timer_add(1.0, _e_mod_cb_config_timer, + _("Fileman Module Configuration data needed upgrading. Your old configuration<br>" + "has been wiped and a new set of defaults initialized. This<br>" + "will happen regularly during development, so don't report a<br>" + "bug. This simply means Fileman module needs new configuration<br>" + "data by default for usable functionality that your old<br>" + "configuration simply lacks. This new set of defaults will fix<br>" + "that by adding it in. You can re-configure things now to your<br>" + "liking. Sorry for the hiccup in your configuration.<br>")); + } + else if (fileman_config->config_version > MOD_CONFIG_FILE_VERSION) + { + _e_mod_fileman_config_free(); + ecore_timer_add(1.0, _e_mod_cb_config_timer, + _("Your Fileman Module configuration is NEWER than Fileman Module version. This is very<br>" + "strange. This should not happen unless you downgraded<br>" + "the Fileman Module or copied the configuration from a place where<br>" + "a newer version of the Fileman Module was running. This is bad and<br>" + "as a precaution your configuration has been now restored to<br>" + "defaults. Sorry for the inconvenience.<br>")); + } + } + if (!fileman_config) { fileman_config = E_NEW(Config, 1); - fileman_config->view.mode = E_FM2_VIEW_MODE_GRID_ICONS; - fileman_config->view.open_dirs_in_place = 0; - fileman_config->view.selector = 0; - fileman_config->view.single_click = 0; - fileman_config->view.no_subdir_jump = 0; - fileman_config->view.show_full_path = 0; - fileman_config->view.show_desktop_icons = 1; - fileman_config->icon.icon.w = 48; - fileman_config->icon.icon.h = 48; - fileman_config->icon.fixed.w = 0; - fileman_config->icon.fixed.h = 0; - fileman_config->icon.extension.show = 1; - fileman_config->list.sort.no_case = 1; - fileman_config->list.sort.dirs.first = 1; - fileman_config->list.sort.dirs.last = 0; - fileman_config->selection.single = 0; - fileman_config->selection.windows_modifiers = 0; + fileman_config->config_version = (MOD_CONFIG_FILE_EPOCH << 16); } +#define IFMODCFG(v) \ + if ((fileman_config->config_version & 0xffff) < (v)) { +#define IFMODCFGEND } + + IFMODCFG(0x0001); + fileman_config->view.mode = E_FM2_VIEW_MODE_GRID_ICONS; + fileman_config->view.open_dirs_in_place = 0; + fileman_config->view.selector = 0; + fileman_config->view.single_click = 0; + fileman_config->view.no_subdir_jump = 0; + fileman_config->view.show_full_path = 0; + fileman_config->view.show_desktop_icons = 1; + fileman_config->icon.icon.w = 48; + fileman_config->icon.icon.h = 48; + fileman_config->icon.fixed.w = 0; + fileman_config->icon.fixed.h = 0; + fileman_config->icon.extension.show = 1; + fileman_config->list.sort.no_case = 1; + fileman_config->list.sort.dirs.first = 1; + fileman_config->list.sort.dirs.last = 0; + fileman_config->selection.single = 0; + fileman_config->selection.windows_modifiers = 0; +IFMODCFGEND; + /* UCHAR's give nasty compile warnings about comparisons so not gonna limit those */ E_CONFIG_LIMIT(fileman_config->view.mode, E_FM2_VIEW_MODE_ICONS, E_FM2_VIEW_MODE_LIST); E_CONFIG_LIMIT(fileman_config->icon.icon.w, 16, 256); E_CONFIG_LIMIT(fileman_config->icon.icon.h, 16, 256); E_CONFIG_LIMIT(fileman_config->icon.list.w, 16, 256); E_CONFIG_LIMIT(fileman_config->icon.list.h, 16, 256); +} + +static void +_e_mod_fileman_config_free(void) +{ + if (fileman_config->theme.background) + evas_stringshare_del(fileman_config->theme.background); + if (fileman_config->theme.frame) + evas_stringshare_del(fileman_config->theme.frame); + if (fileman_config->theme.icons) + evas_stringshare_del(fileman_config->theme.icons); + E_FREE(fileman_config); +} + +static int +_e_mod_cb_config_timer(void *data) +{ + e_util_dialog_show(_("Fileman Configuration Updated"), data); + return 0; } =================================================================== RCS file: /cvs/e/e17/apps/e/src/modules/fileman/e_mod_main.h,v retrieving revision 1.2 retrieving revision 1.3 diff -u -3 -r1.2 -r1.3 --- e_mod_main.h 12 Aug 2007 21:10:02 -0000 1.2 +++ e_mod_main.h 13 Aug 2007 05:05:26 -0000 1.3 @@ -4,6 +4,12 @@ #ifndef E_MOD_MAIN_H #define E_MOD_MAIN_H +/* Increment for Major Changes */ +#define MOD_CONFIG_FILE_EPOCH 0x0001 +/* Increment for Minor Changes (ie: user doesn't need a new config) */ +#define MOD_CONFIG_FILE_GENERATION 0x0001 +#define MOD_CONFIG_FILE_VERSION ((MOD_CONFIG_FILE_EPOCH << 16) | MOD_CONFIG_FILE_GENERATION) + typedef struct _Config Config; #define E_TYPEDEFS 1 @@ -14,7 +20,8 @@ struct _Config { - /* general view mode */ + int config_version; + struct { E_Fm2_View_Mode mode; unsigned char open_dirs_in_place; ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ enlightenment-cvs mailing list enlightenment-cvs@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs