From 7676dae3e0f279d4b193f1c9abf8977c8f52e5d2 Mon Sep 17 00:00:00 2001 From: Alexander Levin Date: Fri, 2 Mar 2012 20:33:13 +0100 Subject: [PATCH] More meaningful name and type for 'root menu' setting The patch changes 'int root_menu' to 'bool root_menu_customized' which better expresses what the value means. Change-Id: Ic272ac161fc05f5d3298e1e46578977dd590ebaf --- apps/root_menu.c | 6 +++--- apps/settings.h | 4 ++-- apps/settings_list.c | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/apps/root_menu.c b/apps/root_menu.c index 6e3164cf04..3f2e933461 100644 --- a/apps/root_menu.c +++ b/apps/root_menu.c @@ -549,7 +549,7 @@ void root_menu_load_from_cfg(void* setting, char *value) if (!main_menu_added) root_menu__[menu_item_count++] = (struct menu_item_ex *)&menu_; root_menu_.flags |= MENU_ITEM_COUNT(menu_item_count); - *(int*)setting = 1; + *(bool*)setting = true; } char* root_menu_write_to_cfg(void* setting, char*buf, int buf_len) @@ -586,13 +586,13 @@ void root_menu_set_default(void* setting, void* defaultval) root_menu__[i] = (struct menu_item_ex *)menu_table[i].item; } root_menu_.flags |= MENU_ITEM_COUNT(MAX_MENU_ITEMS); - *(int*)setting = 0; + *(bool*)setting = false; } bool root_menu_is_changed(void* setting, void* defaultval) { (void)defaultval; - return *(int*)setting != 0; + return *(bool*)setting; } static int item_callback(int action, const struct menu_item_ex *this_item) diff --git a/apps/settings.h b/apps/settings.h index a55ab0c8b6..012e289d0c 100644 --- a/apps/settings.h +++ b/apps/settings.h @@ -823,8 +823,8 @@ struct user_settings #endif char start_directory[MAX_PATHNAME+1]; - /* status setting for the root menu customisability. 0 = default, 1 = loaded from cfg */ - int root_menu; + /* Has the root been customized from the .cfg file? false = no, true = loaded from cfg */ + bool root_menu_customized; }; /** global variables **/ diff --git a/apps/settings_list.c b/apps/settings_list.c index 27cece5550..2258582a78 100644 --- a/apps/settings_list.c +++ b/apps/settings_list.c @@ -1921,7 +1921,7 @@ const struct settings_list settings[] = { "resume rewind", UNIT_SEC, 0, 60, 5, NULL, NULL, NULL), #endif - CUSTOM_SETTING(0, root_menu, + CUSTOM_SETTING(0, root_menu_customized, LANG_ROCKBOX_TITLE, /* lang string here is never actually used */ NULL, "root menu order", root_menu_load_from_cfg, root_menu_write_to_cfg,