From d43d211a5830eceb2b667141d65d0488e7790769 Mon Sep 17 00:00:00 2001 From: Thomas Martitz Date: Sat, 20 Feb 2010 21:32:31 +0000 Subject: [PATCH] Factor out some drawing code. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@24796 a1c6a512-1295-4272-9138-f99709370657 --- apps/recorder/radio.c | 68 +++++++++++++++---------------------------- 1 file changed, 23 insertions(+), 45 deletions(-) diff --git a/apps/recorder/radio.c b/apps/recorder/radio.c index 7eed961202..9e891389bd 100644 --- a/apps/recorder/radio.c +++ b/apps/recorder/radio.c @@ -455,6 +455,19 @@ static void talk_preset(int preset, bool fallback, bool enqueue) } } +static void fms_restore(struct viewport vp[NB_SCREENS]) +{ + struct screen *display; + int i; + FOR_NB_SCREENS(i) + { + display = &screens[i]; + display->set_viewport(&vp[i]); + display->clear_viewport(); + display->update_viewport(); + } +} + int radio_screen(void) { char buf[MAX_PATH]; @@ -502,11 +515,8 @@ int radio_screen(void) if (global_settings.buttonbar) vp[i].height -= BUTTONBAR_HEIGHT; #endif - screens[i].set_viewport(&vp[i]); - screens[i].stop_scroll(); - screens[i].clear_viewport(); - screens[i].update_viewport(); } + fms_restore(vp); fh = font_get(FONT_UI)->height; @@ -754,13 +764,7 @@ int radio_screen(void) } radio_menu(); curr_preset = find_preset(curr_freq); - FOR_NB_SCREENS(i) - { - screens[i].set_viewport(&vp[i]); - screens[i].clear_viewport(); - screens[i].update_viewport(); - screens[i].set_viewport(NULL); - } + fms_restore(vp); #ifdef HAVE_BUTTONBAR gui_buttonbar_set(&buttonbar, str(LANG_BUTTONBAR_MENU), str(LANG_PRESET), @@ -775,25 +779,14 @@ int radio_screen(void) { splash(HZ, ID2P(LANG_FM_NO_PRESETS)); update_screen = true; - FOR_NB_SCREENS(i) - { - screens[i].set_viewport(&vp[i]); - screens[i].clear_viewport(); - screens[i].update_viewport(); - screens[i].set_viewport(NULL); - } + fms_restore(vp); break; } - handle_radio_presets(); FOR_NB_SCREENS(i) - { - screens[i].set_viewport(&vp[i]); - screens[i].stop_scroll(); - screens[i].clear_viewport(); - screens[i].update_viewport(); - screens[i].set_viewport(NULL); - } + screens[i].scroll_stop(&vp[i]); + handle_radio_presets(); + fms_restore(vp); #ifdef HAVE_BUTTONBAR gui_buttonbar_set(&buttonbar, str(LANG_BUTTONBAR_MENU), @@ -812,14 +805,7 @@ int radio_screen(void) done = true; break; } - FOR_NB_SCREENS(i) - { - screens[i].set_viewport(&vp[i]); - screens[i].stop_scroll(); - screens[i].clear_viewport(); - screens[i].update_viewport(); - screens[i].set_viewport(NULL); - } + fms_restore(vp); update_screen = true; } break; @@ -940,7 +926,6 @@ int radio_screen(void) peak_meter_screen(&screens[i],0, fh*(top_of_screen + 4),fh); screens[i].update_rect(0, fh*(top_of_screen + 4), screens[i].getwidth(), fh); - screens[i].set_viewport(NULL); } } @@ -1039,11 +1024,7 @@ int radio_screen(void) #endif /* CONFIG_CODEC != SWCODEC */ FOR_NB_SCREENS(i) - { screens[i].update_viewport(); - screens[i].set_viewport(NULL); - } - #ifdef HAVE_BUTTONBAR gui_buttonbar_draw(&buttonbar); #endif @@ -1087,12 +1068,7 @@ int radio_screen(void) if(audio_status() & AUDIO_STATUS_ERROR) { splash(0, str(LANG_DISK_FULL)); - FOR_NB_SCREENS(i) - { - screens[i].set_viewport(&vp[i]); - screens[i].update_viewport(); - screens[i].set_viewport(NULL); - } + fms_restore(vp); audio_error_clear(); while(1) @@ -1138,6 +1114,7 @@ int radio_screen(void) FOR_NB_SCREENS(i) { screens[i].scroll_stop(&vp[i]); + screens[i].set_viewport(NULL); } in_screen = false; #if CONFIG_CODEC != SWCODEC @@ -1489,6 +1466,7 @@ static int handle_radio_presets(void) result = 2; } } + gui_synclist_scroll_stop(&lists); return result - 1; }