Another wps rework:

*rename gui_wps_refresh() to gui_wps_redraw() and update() to gui_wps_update() to better describe what they do
*Clear things up and differentiate better between gui_wps_redraw()/gui_wps_display()/gui_wps_update() (it wasn't so what they're supposed to do before)
*cleanup gui_wps_display() and gui_wps_update() quite a bit
*Remove unused/unneeded code, do some pointer copying to save binsize, and code cleanup at several places

Visible changes should be small

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@20483 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 15 insertions, 4 deletions

diff --git a/apps/gui/gwps-common.h b/apps/gui/gwps-common.h
index 25ce435592..8752c1d162 100644
--- a/apps/gui/gwps-common.h
+++ b/apps/gui/gwps-common.h
@@ -25,14 +25,25 @@
 
 #include "gwps.h"
 
+/* fades the volume, e.g. on pause or stop */
 void fade(bool fade_in, bool updatewps);
-bool gui_wps_display(void);
+
+/* Initially display the wps, can fall back to the build-in wps
+ * if the chosen wps is invalid */
+bool gui_wps_display(struct gui_wps *gui_wps);
+
+/* return true if screen restore is needed
+   return false otherwise */
 bool update_onvol_change(struct gui_wps * gwps);
-bool update(struct gui_wps *gwps);
+
+/* Update track info related stuff, handles cue sheets as well, and redraw */
+bool gui_wps_update(struct gui_wps *gwps);
+
 bool ffwd_rew(int button);
 void display_keylock_text(bool locked);
 
-bool gui_wps_refresh(struct gui_wps *gwps,
+/* Refresh the WPS according to refresh_mode. */
+bool gui_wps_redraw(struct gui_wps *gwps,
                      int ffwd_offset,
-                     unsigned char refresh_mode);
+                     unsigned refresh_mode);
 #endif