From 843c7efaf8c953fc3bec40a7da0be3a5da9950df Mon Sep 17 00:00:00 2001 From: Jonathan Gordon Date: Mon, 6 Apr 2009 00:39:43 +0000 Subject: FS9795 - some playback cleanup. * Use events to notify things when the track has changed instead of the nasty has_track_changed() * Event for when the mp3entry for the next track is avilable (which allows alot more tags to be static which means less redrawing in the WPS) * virtually guarentee that the mp3entry sturct returned by audio_current/next_track() is going to be valid for the duration of the current track. The only time it wont be now is during the time between the codec finishing the previous track and the next track actually starting (~2s), but this is not an issue as long as it is called again when the TRACK_CHANGED event happens (or just use the pointer that gives) It is still possible to confuse the WPS with the next tracks id3 info being displayed but this should fix itself up faster than it used to (and be harder to do) git-svn-id: svn://svn.rockbox.org/rockbox/trunk@20633 a1c6a512-1295-4272-9138-f99709370657 --- apps/mpeg.c | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'apps/mpeg.c') diff --git a/apps/mpeg.c b/apps/mpeg.c index cde72ab54f..5ebf58fcf0 100644 --- a/apps/mpeg.c +++ b/apps/mpeg.c @@ -852,6 +852,7 @@ static void transfer_end(unsigned char** ppbuf, size_t* psize) static struct trackdata *add_track_to_tag_list(const char *filename) { struct trackdata *track; + bool send_nid3_event; if(num_tracks_in_memory() >= MAX_TRACK_ENTRIES) { @@ -882,7 +883,11 @@ static struct trackdata *add_track_to_tag_list(const char *filename) if (cuesheet_callback(filename)) track->id3.cuesheet_type = 1; + /* if this track is the next track then let the UI know it can get it */ + send_nid3_event = (track_write_idx == track_read_idx + 1); track_write_idx = (track_write_idx+1) & MAX_TRACK_ENTRIES_MASK; + if (send_nid3_event) + send_event(PLAYBACK_EVENT_NEXTTRACKID3_AVAILABLE, NULL); debug_tags(); return track; } -- cgit v1.2.3