From daf66940b1a06de2681c91dcb8cf72d95a234e86 Mon Sep 17 00:00:00 2001 From: Jonathan Gordon Date: Sat, 17 Mar 2007 12:33:34 +0000 Subject: Unify the way functions are called from menus. Optionally, the functions return value can be checked for a value to tell the menu to quit. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@12821 a1c6a512-1295-4272-9138-f99709370657 --- apps/menus/display_menu.c | 44 +++++++++++++++++----------------- apps/menus/eq_menu.c | 57 ++++++++++++++++++++++++--------------------- apps/menus/main_menu.c | 49 ++++++++++++++++++++------------------ apps/menus/playlist_menu.c | 18 +++++++------- apps/menus/recording_menu.c | 41 ++++++++++++++++---------------- apps/menus/settings_menu.c | 35 ++++++++++++++++------------ 6 files changed, 130 insertions(+), 114 deletions(-) (limited to 'apps/menus') diff --git a/apps/menus/display_menu.c b/apps/menus/display_menu.c index 23915dcfef..3714950223 100644 --- a/apps/menus/display_menu.c +++ b/apps/menus/display_menu.c @@ -149,7 +149,6 @@ static int set_bg_color(void) screens[SCREEN_MAIN].set_background(global_settings.bg_color); return res; } - static int reset_color(void) { global_settings.fg_color = LCD_DEFAULT_FG; @@ -159,14 +158,14 @@ static int reset_color(void) screens[SCREEN_MAIN].set_background(global_settings.bg_color); return 0; } -MENUITEM_FUNCTION(clear_main_bd, ID2P(LANG_CLEAR_BACKDROP), - clear_main_backdrop, NULL, Icon_NOICON); -MENUITEM_FUNCTION(set_bg_col, ID2P(LANG_BACKGROUND_COLOR), - set_bg_color, NULL, Icon_NOICON); -MENUITEM_FUNCTION(set_fg_col, ID2P(LANG_FOREGROUND_COLOR), - set_fg_color, NULL, Icon_NOICON); -MENUITEM_FUNCTION(reset_colors, ID2P(LANG_RESET_COLORS), - reset_color, NULL, Icon_NOICON); +MENUITEM_FUNCTION(clear_main_bd, 0, ID2P(LANG_CLEAR_BACKDROP), + clear_main_backdrop, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(set_bg_col, 0, ID2P(LANG_BACKGROUND_COLOR), + set_bg_color, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(set_fg_col, 0, ID2P(LANG_FOREGROUND_COLOR), + set_fg_color, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(reset_colors, 0, ID2P(LANG_RESET_COLORS), + reset_color, NULL, 0, NULL, Icon_NOICON); #endif /* now the actual menu */ @@ -498,12 +497,12 @@ static int peak_meter_max(void) { settings_apply_pm_range(); return retval; } -MENUITEM_FUNCTION(peak_meter_scale_item, ID2P(LANG_PM_SCALE), - peak_meter_scale, NULL, Icon_NOICON); -MENUITEM_FUNCTION(peak_meter_min_item, ID2P(LANG_PM_MIN), - peak_meter_min, NULL, Icon_NOICON); -MENUITEM_FUNCTION(peak_meter_max_item, ID2P(LANG_PM_MAX), - peak_meter_max, NULL, Icon_NOICON); +MENUITEM_FUNCTION(peak_meter_scale_item, 0, ID2P(LANG_PM_SCALE), + peak_meter_scale, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(peak_meter_min_item, 0, ID2P(LANG_PM_MIN), + peak_meter_min, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(peak_meter_max_item, 0, ID2P(LANG_PM_MAX), + peak_meter_max, NULL, 0, NULL, Icon_NOICON); MAKE_MENU(peak_meter_menu, ID2P(LANG_PM_MENU), NULL, Icon_NOICON, &peak_meter_release, &peak_meter_hold, &peak_meter_clip_hold, @@ -534,14 +533,17 @@ static int browse_folder(void *param) } #ifdef HAVE_LCD_BITMAP -MENUITEM_FUNCTION_WPARAM(browse_fonts, ID2P(LANG_CUSTOM_FONT), - browse_folder, (void*)&fonts, NULL, Icon_NOICON); +MENUITEM_FUNCTION(browse_fonts, MENU_FUNC_USEPARAM, + ID2P(LANG_CUSTOM_FONT), + browse_folder, (void*)&fonts, 0, NULL, Icon_NOICON); #endif -MENUITEM_FUNCTION_WPARAM(browse_wps, ID2P(LANG_WHILE_PLAYING), - browse_folder, (void*)&wps, NULL, Icon_NOICON); +MENUITEM_FUNCTION(browse_wps, MENU_FUNC_USEPARAM, + ID2P(LANG_WHILE_PLAYING), + browse_folder, (void*)&wps, 0, NULL, Icon_NOICON); #ifdef HAVE_REMOTE_LCD -MENUITEM_FUNCTION_WPARAM(browse_rwps, ID2P(LANG_REMOTE_WHILE_PLAYING), - browse_folder, (void*)&rwps, NULL, Icon_NOICON); +MENUITEM_FUNCTION(browse_rwps, MENU_FUNC_USEPARAM, + ID2P(LANG_REMOTE_WHILE_PLAYING), + browse_folder, (void*)&rwps, 0, NULL, Icon_NOICON); #endif MENUITEM_SETTING(show_icons, &global_settings.show_icons, NULL); diff --git a/apps/menus/eq_menu.c b/apps/menus/eq_menu.c index 0cf63bb20d..f2c1f9b22d 100644 --- a/apps/menus/eq_menu.c +++ b/apps/menus/eq_menu.c @@ -142,21 +142,21 @@ MENUITEM_SETTING(gain_2, &global_settings.eq_band2_gain, dsp_set_coefs_callback) MENUITEM_SETTING(gain_3, &global_settings.eq_band3_gain, dsp_set_coefs_callback); MENUITEM_SETTING(gain_4, &global_settings.eq_band4_gain, dsp_set_coefs_callback); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(gain_item_0, do_option, (void*)&gain_0, NULL, - gainitem_get_name, - &global_settings.eq_band0_cutoff, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(gain_item_1, do_option, (void*)&gain_1, NULL, - gainitem_get_name, - &global_settings.eq_band1_cutoff, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(gain_item_2, do_option, (void*)&gain_2, NULL, - gainitem_get_name, - &global_settings.eq_band2_cutoff, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(gain_item_3, do_option, (void*)&gain_3, NULL, - gainitem_get_name, - &global_settings.eq_band3_cutoff, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(gain_item_4, do_option, (void*)&gain_4, NULL, - gainitem_get_name, - &global_settings.eq_band4_cutoff, Icon_NOICON); +MENUITEM_FUNCTION_DYNTEXT(gain_item_0, MENU_FUNC_USEPARAM, do_option, (void*)&gain_0, + 0, gainitem_get_name, &global_settings.eq_band0_cutoff, + NULL, Icon_NOICON); +MENUITEM_FUNCTION_DYNTEXT(gain_item_1, MENU_FUNC_USEPARAM, do_option, (void*)&gain_1, + 0, gainitem_get_name, &global_settings.eq_band1_cutoff, + NULL, Icon_NOICON); +MENUITEM_FUNCTION_DYNTEXT(gain_item_2, MENU_FUNC_USEPARAM, do_option, (void*)&gain_2, + 0, gainitem_get_name, &global_settings.eq_band2_cutoff, + NULL, Icon_NOICON); +MENUITEM_FUNCTION_DYNTEXT(gain_item_3, MENU_FUNC_USEPARAM, do_option, (void*)&gain_3, + 0, gainitem_get_name, &global_settings.eq_band3_cutoff, + NULL, Icon_NOICON); +MENUITEM_FUNCTION_DYNTEXT(gain_item_4, MENU_FUNC_USEPARAM, do_option, (void*)&gain_4, + 0, gainitem_get_name, &global_settings.eq_band4_cutoff, + NULL, Icon_NOICON); MAKE_MENU(gain_menu, ID2P(LANG_EQUALIZER_GAIN), NULL, Icon_NOICON, &gain_item_0, &gain_item_1, &gain_item_2, &gain_item_3, &gain_item_4); @@ -192,12 +192,15 @@ int do_center_band_menu(void* param) } MAKE_MENU(band_0_menu, ID2P(LANG_EQUALIZER_BAND_LOW_SHELF), NULL, Icon_EQ, &cutoff_0, &q_0, &gain_0); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(band_1_menu, do_center_band_menu, (void*)1, NULL, - centerband_get_name, (void*)1, Icon_EQ); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(band_2_menu, do_center_band_menu, (void*)2, NULL, - centerband_get_name, (void*)2, Icon_EQ); -MENUITEM_FUNCTION_WPARAM_DYNTEXT(band_3_menu, do_center_band_menu, (void*)3, NULL, - centerband_get_name, (void*)3, Icon_EQ); +MENUITEM_FUNCTION_DYNTEXT(band_1_menu, MENU_FUNC_USEPARAM, + do_center_band_menu, (void*)1, 0, + centerband_get_name, (void*)1, NULL, Icon_EQ); +MENUITEM_FUNCTION_DYNTEXT(band_2_menu, MENU_FUNC_USEPARAM, + do_center_band_menu, (void*)2, 0, + centerband_get_name, (void*)2, NULL, Icon_EQ); +MENUITEM_FUNCTION_DYNTEXT(band_3_menu, MENU_FUNC_USEPARAM, + do_center_band_menu, (void*)3, 0, + centerband_get_name, (void*)3, NULL, Icon_EQ); MAKE_MENU(band_4_menu, ID2P(LANG_EQUALIZER_BAND_HIGH_SHELF), NULL, Icon_EQ, &cutoff_4, &q_4, &gain_4); @@ -625,13 +628,13 @@ bool eq_browse_presets(void) } -MENUITEM_FUNCTION(eq_graphical, ID2P(LANG_EQUALIZER_GRAPHICAL), - (int(*)(void))eq_menu_graphical, NULL, +MENUITEM_FUNCTION(eq_graphical, 0, ID2P(LANG_EQUALIZER_GRAPHICAL), + (int(*)(void))eq_menu_graphical, NULL, 0, NULL, Icon_EQ); -MENUITEM_FUNCTION(eq_save, ID2P(LANG_EQUALIZER_SAVE), - (int(*)(void))eq_save_preset, NULL, Icon_NOICON); -MENUITEM_FUNCTION(eq_browse, ID2P(LANG_EQUALIZER_BROWSE), - (int(*)(void))eq_browse_presets, NULL, Icon_NOICON); +MENUITEM_FUNCTION(eq_save, 0, ID2P(LANG_EQUALIZER_SAVE), + (int(*)(void))eq_save_preset, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(eq_browse, 0, ID2P(LANG_EQUALIZER_BROWSE), + (int(*)(void))eq_browse_presets, NULL, 0, NULL, Icon_NOICON); MAKE_MENU(equalizer_menu, ID2P(LANG_EQUALIZER), NULL, Icon_EQ, &eq_enable, &eq_graphical, &eq_precut, &gain_menu, diff --git a/apps/menus/main_menu.c b/apps/menus/main_menu.c index bd7b35dea2..b91b2f5553 100644 --- a/apps/menus/main_menu.c +++ b/apps/menus/main_menu.c @@ -95,13 +95,14 @@ static int write_settings_file(void* param) return settings_save_config((intptr_t)param); } -MENUITEM_FUNCTION_WPARAM(browse_configs, ID2P(LANG_CUSTOM_CFG), - browse_folder, (void*)&config, NULL, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM(save_settings_item, ID2P(LANG_SAVE_SETTINGS), - write_settings_file, (void*)SETTINGS_SAVE_ALL, NULL, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM(save_theme_item, ID2P(LANG_SAVE_THEME), - write_settings_file, (void*)SETTINGS_SAVE_THEME, NULL, Icon_NOICON); -MENUITEM_FUNCTION(reset_settings_item,ID2P(LANG_RESET),reset_settings, NULL, Icon_NOICON); +MENUITEM_FUNCTION(browse_configs, MENU_FUNC_USEPARAM, ID2P(LANG_CUSTOM_CFG), + browse_folder, (void*)&config, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(save_settings_item, MENU_FUNC_USEPARAM, ID2P(LANG_SAVE_SETTINGS), + write_settings_file, (void*)SETTINGS_SAVE_ALL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(save_theme_item, MENU_FUNC_USEPARAM, ID2P(LANG_SAVE_THEME), + write_settings_file, (void*)SETTINGS_SAVE_THEME, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(reset_settings_item, 0, ID2P(LANG_RESET), + reset_settings, NULL, 0, NULL, Icon_NOICON); MAKE_MENU(manage_settings, ID2P(LANG_MANAGE_MENU), NULL, Icon_Config, &browse_configs, &reset_settings_item, @@ -339,8 +340,8 @@ static bool show_info(void) action_signalscreenchange(); return false; } -MENUITEM_FUNCTION(show_info_item, ID2P(LANG_INFO_MENU), - (menu_function)show_info, NULL, Icon_NOICON); +MENUITEM_FUNCTION(show_info_item, 0, ID2P(LANG_INFO_MENU), + (menu_function)show_info, NULL, 0, NULL, Icon_NOICON); /* sleep Menu */ @@ -372,21 +373,23 @@ static int sleep_timer(void) &sleep_timer_set, -5, 300, 0, sleep_timer_formatter); } -MENUITEM_FUNCTION(sleep_timer_call, ID2P(LANG_SLEEP_TIMER), sleep_timer, - NULL, Icon_Menu_setting); /* make it look like a +MENUITEM_FUNCTION(sleep_timer_call, 0, ID2P(LANG_SLEEP_TIMER), sleep_timer, + NULL, 0, NULL, Icon_Menu_setting); /* make it look like a setting to the user */ -MENUITEM_FUNCTION(show_credits_item, ID2P(LANG_VERSION), - (menu_function)show_credits, NULL, Icon_NOICON); -MENUITEM_FUNCTION(show_runtime_item, ID2P(LANG_RUNNING_TIME), - (menu_function)view_runtime, NULL, Icon_NOICON); -MENUITEM_FUNCTION(debug_menu_item, ID2P(LANG_DEBUG), - (menu_function)debug_menu, NULL, Icon_NOICON); +MENUITEM_FUNCTION(show_credits_item, 0, ID2P(LANG_VERSION), + (menu_function)show_credits, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(show_runtime_item, 0, ID2P(LANG_RUNNING_TIME), + (menu_function)view_runtime, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(debug_menu_item, 0, ID2P(LANG_DEBUG), + (menu_function)debug_menu, NULL, 0, NULL, Icon_NOICON); #ifdef SIMULATOR -MENUITEM_FUNCTION(simulate_usb_item, ID2P(LANG_USB), - (menu_function)simulate_usb, NULL, Icon_NOICON); +MENUITEM_FUNCTION(simulate_usb_item, 0, ID2P(LANG_USB), + (menu_function)simulate_usb, NULL, 0, NULL, Icon_NOICON); #ifdef ROCKBOX_HAS_LOGF -MENUITEM_FUNCTION(logfdisplay_item, "logf",(int (*)(void)) logfdisplay, NULL, Icon_NOICON); -MENUITEM_FUNCTION(logfdump_item, "logfdump",(int (*)(void)) logfdump, NULL, Icon_NOICON); +MENUITEM_FUNCTION(logfdisplay_item, 0, "logf", + (int (*)(void)) logfdisplay, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(logfdump_item, 0, "logfdump", + (int (*)(void)) logfdump, NULL, 0, NULL, Icon_NOICON); #endif #endif @@ -406,8 +409,8 @@ MAKE_MENU(info_menu, ID2P(LANG_INFO), 0, Icon_Questionmark, /***********************************/ /* MAIN MENU */ -MENUITEM_FUNCTION_WPARAM(browse_themes, ID2P(LANG_CUSTOM_THEME), - browse_folder, (void*)&theme, NULL, Icon_Folder); +MENUITEM_FUNCTION(browse_themes, MENU_FUNC_USEPARAM, ID2P(LANG_CUSTOM_THEME), + browse_folder, (void*)&theme, 0, NULL, Icon_Folder); #ifdef HAVE_LCD_CHARCELLS int mainmenu_callback(int action,const struct menu_item_ex *this_item) diff --git a/apps/menus/playlist_menu.c b/apps/menus/playlist_menu.c index d4b311154a..b73341c1d9 100644 --- a/apps/menus/playlist_menu.c +++ b/apps/menus/playlist_menu.c @@ -62,14 +62,16 @@ int save_playlist_screen(struct playlist_info* playlist) return 0; } -MENUITEM_FUNCTION(create_playlist_item, ID2P(LANG_CREATE_PLAYLIST), - (int(*)(void))create_playlist, NULL, Icon_NOICON); -MENUITEM_FUNCTION(view_playlist, ID2P(LANG_VIEW_DYNAMIC_PLAYLIST), - (int(*)(void))playlist_viewer, NULL, Icon_NOICON); -MENUITEM_FUNCTION_WPARAM(save_playlist, ID2P(LANG_SAVE_DYNAMIC_PLAYLIST), - (int(*)(void*))save_playlist_screen, NULL, NULL, Icon_NOICON); -MENUITEM_FUNCTION(catalog, ID2P(LANG_CATALOG), - (int(*)(void))catalog_view_playlists, NULL, Icon_NOICON); +MENUITEM_FUNCTION(create_playlist_item, 0, ID2P(LANG_CREATE_PLAYLIST), + (int(*)(void))create_playlist, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(view_playlist, 0, ID2P(LANG_VIEW_DYNAMIC_PLAYLIST), + (int(*)(void))playlist_viewer, NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(save_playlist, MENU_FUNC_USEPARAM, ID2P(LANG_SAVE_DYNAMIC_PLAYLIST), + (int(*)(void*))save_playlist_screen, + NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(catalog, 0, ID2P(LANG_CATALOG), + (int(*)(void))catalog_view_playlists, + NULL, 0, NULL, Icon_NOICON); MENUITEM_SETTING(recursive_dir_insert, &global_settings.recursive_dir_insert, NULL); MENUITEM_SETTING(warn_on_erase, &global_settings.warnon_erase_dynplaylist, NULL); diff --git a/apps/menus/recording_menu.c b/apps/menus/recording_menu.c index 76763b101d..43bd8a781f 100644 --- a/apps/menus/recording_menu.c +++ b/apps/menus/recording_menu.c @@ -90,8 +90,8 @@ static int recsource_func(void) &global_settings.rec_source, INT, names, n_opts, NULL ); } -MENUITEM_FUNCTION(recsource, ID2P(LANG_RECORDING_SOURCE), - recsource_func, recmenu_callback, Icon_Menu_setting); +MENUITEM_FUNCTION(recsource, 0, ID2P(LANG_RECORDING_SOURCE), + recsource_func, NULL, 0, recmenu_callback, Icon_Menu_setting); #if CONFIG_CODEC == SWCODEC /* Makes an options list from a source list of options and indexes */ @@ -205,8 +205,8 @@ static int recfrequency_func(void) return ret; #endif /* CONFIG_CODEC == SWCODEC */ } /* recfrequency */ -MENUITEM_FUNCTION(recfrequency, ID2P(LANG_RECORDING_FREQUENCY), - recfrequency_func, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(recfrequency, 0, ID2P(LANG_RECORDING_FREQUENCY), + recfrequency_func, NULL, 0, NULL, Icon_Menu_setting); static int recchannels_func(void) @@ -253,8 +253,8 @@ static int recchannels_func(void) return ret; #endif /* CONFIG_CODEC == SWCODEC */ } -MENUITEM_FUNCTION(recchannels, ID2P(LANG_RECORDING_CHANNELS), - recchannels_func, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(recchannels, 0, ID2P(LANG_RECORDING_CHANNELS), + recchannels_func, NULL, 0, NULL, Icon_Menu_setting); #if CONFIG_CODEC == SWCODEC @@ -279,11 +279,12 @@ static int recformat_func(void) return res; } /* recformat */ -MENUITEM_FUNCTION(recformat, ID2P(LANG_RECORDING_FORMAT), - recformat_func, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(recformat, 0, ID2P(LANG_RECORDING_FORMAT), + recformat_func, NULL, 0, NULL, Icon_Menu_setting); -MENUITEM_FUNCTION(enc_global_config_menu_item, ID2P(LANG_ENCODER_SETTINGS), - (int(*)(void))enc_global_config_menu, NULL, Icon_Submenu); +MENUITEM_FUNCTION(enc_global_config_menu_item, 0, ID2P(LANG_ENCODER_SETTINGS), + (int(*)(void))enc_global_config_menu, + NULL, 0, NULL, Icon_Submenu); #endif /* CONFIG_CODEC == SWCODEC */ @@ -323,8 +324,8 @@ static int recdirectory_func(void) &global_settings.rec_directory, INT, names, 2, NULL ); } -MENUITEM_FUNCTION(recdirectory, ID2P(LANG_RECORD_DIRECTORY), - recdirectory_func, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(recdirectory, 0, ID2P(LANG_RECORD_DIRECTORY), + recdirectory_func, NULL, 0, NULL, Icon_Menu_setting); MENUITEM_SETTING(cliplight, &global_settings.cliplight, NULL); @@ -362,10 +363,10 @@ static int agc_cliptime_func(void) &global_settings.rec_agc_cliptime, INT, names, 5, NULL ); } -MENUITEM_FUNCTION(agc_preset, ID2P(LANG_RECORD_AGC_PRESET), - agc_preset_func, NULL, Icon_Menu_setting); -MENUITEM_FUNCTION(agc_cliptime, ID2P(LANG_RECORD_AGC_CLIPTIME), - agc_cliptime_func, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(agc_preset, 0, ID2P(LANG_RECORD_AGC_PRESET), + agc_preset_func, NULL, 0, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(agc_cliptime, 0, ID2P(LANG_RECORD_AGC_CLIPTIME), + agc_cliptime_func, NULL, 0, NULL, Icon_Menu_setting); #endif /* HAVE_AGC */ /** Rec trigger **/ @@ -796,8 +797,8 @@ bool rectrigger(void) return retval; } -MENUITEM_FUNCTION(rectrigger_item, ID2P(LANG_RECORD_TRIGGER), - (int(*)(void))rectrigger, NULL, Icon_Menu_setting); +MENUITEM_FUNCTION(rectrigger_item, 0, ID2P(LANG_RECORD_TRIGGER), + (int(*)(void))rectrigger, NULL, 0, NULL, Icon_Menu_setting); @@ -836,5 +837,5 @@ bool recording_menu(bool no_source) return do_menu(&recording_setting_menu, NULL) == MENU_ATTACHED_USB; }; -MENUITEM_FUNCTION_WPARAM(recording_settings, ID2P(LANG_RECORDING_SETTINGS), - (int (*)(void*))recording_menu,0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(recording_settings, MENU_FUNC_USEPARAM, ID2P(LANG_RECORDING_SETTINGS), + (int (*)(void*))recording_menu, 0, 0, NULL, Icon_NOICON); diff --git a/apps/menus/settings_menu.c b/apps/menus/settings_menu.c index d150aac148..ace17a2055 100644 --- a/apps/menus/settings_menu.c +++ b/apps/menus/settings_menu.c @@ -62,15 +62,19 @@ static void tagcache_update_with_splash(void) MENUITEM_SETTING(tagcache_ram, &global_settings.tagcache_ram, NULL); #endif MENUITEM_SETTING(tagcache_autoupdate, &global_settings.tagcache_autoupdate, NULL); -MENUITEM_FUNCTION(tc_init, ID2P(LANG_TAGCACHE_FORCE_UPDATE), - (int(*)(void))tagcache_rebuild_with_splash, NULL, Icon_NOICON); -MENUITEM_FUNCTION(tc_update, ID2P(LANG_TAGCACHE_UPDATE), - (int(*)(void))tagcache_update_with_splash, NULL, Icon_NOICON); +MENUITEM_FUNCTION(tc_init, 0, ID2P(LANG_TAGCACHE_FORCE_UPDATE), + (int(*)(void))tagcache_rebuild_with_splash, + NULL, 0, NULL, Icon_NOICON); +MENUITEM_FUNCTION(tc_update, 0, ID2P(LANG_TAGCACHE_UPDATE), + (int(*)(void))tagcache_update_with_splash, + NULL, 0, NULL, Icon_NOICON); MENUITEM_SETTING(runtimedb, &global_settings.runtimedb, NULL); -MENUITEM_FUNCTION(tc_export, ID2P(LANG_TAGCACHE_EXPORT), - (int(*)(void))tagtree_export, NULL, Icon_NOICON); -MENUITEM_FUNCTION(tc_import, ID2P(LANG_TAGCACHE_IMPORT), - (int(*)(void))tagtree_import, NULL, Icon_NOICON); +MENUITEM_FUNCTION(tc_export, 0, ID2P(LANG_TAGCACHE_EXPORT), + (int(*)(void))tagtree_export, NULL, 0, + NULL, Icon_NOICON); +MENUITEM_FUNCTION(tc_import, 0, ID2P(LANG_TAGCACHE_IMPORT), + (int(*)(void))tagtree_import, NULL, 0, + NULL, Icon_NOICON); MAKE_MENU(tagcache_menu, ID2P(LANG_TAGCACHE), 0, Icon_NOICON, #ifdef HAVE_TC_RAMCACHE &tagcache_ram, @@ -216,7 +220,8 @@ static int timedate_set(void) return result; } -MENUITEM_FUNCTION(time_set, ID2P(LANG_TIME), timedate_set, NULL, Icon_NOICON); +MENUITEM_FUNCTION(time_set, 0, ID2P(LANG_TIME), + timedate_set, NULL, 0, NULL, Icon_NOICON); MENUITEM_SETTING(timeformat, &global_settings.timeformat, NULL); MAKE_MENU(time_menu, ID2P(LANG_TIME_MENU), 0, Icon_NOICON, &time_set, &timeformat); #endif @@ -225,8 +230,8 @@ MAKE_MENU(time_menu, ID2P(LANG_TIME_MENU), 0, Icon_NOICON, &time_set, &timeforma MENUITEM_SETTING(poweroff, &global_settings.poweroff, NULL); #ifdef HAVE_RTC_ALARM -MENUITEM_FUNCTION(alarm_screen_call, ID2P(LANG_ALARM_MOD_ALARM_MENU), - (menu_function)alarm_screen, NULL, Icon_NOICON); +MENUITEM_FUNCTION(alarm_screen_call, 0, ID2P(LANG_ALARM_MOD_ALARM_MENU), + (menu_function)alarm_screen, NULL, 0, NULL, Icon_NOICON); #if CONFIG_TUNER || defined(HAVE_RECORDING) #if CONFIG_TUNER && !defined(HAVE_RECORDING) @@ -275,8 +280,8 @@ static int alarm_setting(void) INT, items, i, NULL); } -MENUITEM_FUNCTION(alarm_wake_up_screen, ID2P(LANG_ALARM_WAKEUP_SCREEN), - alarm_setting, alarm_callback, Icon_Menu_setting); +MENUITEM_FUNCTION(alarm_wake_up_screen, 0, ID2P(LANG_ALARM_WAKEUP_SCREEN), + alarm_setting, NULL, 0, alarm_callback, Icon_Menu_setting); #endif /* CONFIG_TUNER || defined(HAVE_RECORDING) */ #endif /* HAVE_RTC_ALARM */ @@ -409,8 +414,8 @@ static int language_browse(void) { return (int)rockbox_browse(LANG_DIR, SHOW_LNG); } -MENUITEM_FUNCTION(browse_langs, ID2P(LANG_LANGUAGE), language_browse, - NULL, Icon_Language); +MENUITEM_FUNCTION(browse_langs, 0, ID2P(LANG_LANGUAGE), language_browse, + NULL, 0, NULL, Icon_Language); MAKE_MENU(settings_menu_item, ID2P(LANG_GENERAL_SETTINGS), 0, Icon_General_settings_menu, -- cgit v1.2.3