Nyaaori/rockbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix FS#2687 (Rewinding at end of song confuses playback) by making the codec skip back if the user seeks during the transition window. This also completely fixes the small glitch in the progressbar when a user skipped back to a buffered song.

Browse source 
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@16025 a1c6a512-1295-4272-9138-f99709370657
This commit is contained in:
Nicolas Pennequin 2008-01-08 14:29:11 +00:00
parent 397296af28
commit 3d7f64c615
1 changed files with 14 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
apps/playback.c
View file

@ -1200,9 +1200,6 @@ static bool codec_load_next_track(void)
prev_track_elapsed = curtrack_id3.elapsed;
if (ci.seek_time)
codec_seek_complete_callback();
#ifdef AB_REPEAT_ENABLE
ab_end_of_track_report();
#endif
@ -1254,11 +1251,6 @@ static bool codec_request_next_track_callback(void)
if (!codec_load_next_track())
return false;
/* Seek to the beginning of the new track because if the struct mp3entry was
buffered, "elapsed" might not be zero (if the track has been played
already but not unbuffered) */
codec_seek_buffer_callback(curtrack_id3.first_frame_offset);
/* Check if the next codec is the same file. */
if (prev_codectype == get_codec_base_type(curtrack_id3.codectype))
{
@ -2345,6 +2337,13 @@ static void audio_finalise_track_change(void)
clear_track_info(prev_ti);
}
if (prev_ti && prev_ti->id3_hid >= 0)
{
/* Reset the elapsed time to force the progressbar to be empty if
the user skips back to this track */
bufgetid3(prev_ti->id3_hid)->elapsed = 0;
}
if (track_changed_callback)
track_changed_callback(&curtrack_id3);
@ -2473,6 +2472,13 @@ static void audio_thread(void)
LOGFQUEUE("audio < Q_AUDIO_FF_REWIND");
if (!playing)
break;
if (automatic_skip)
{
/* An automatic track skip is in progress. Finalize it,
then go back to the previous track */
audio_finalise_track_change();
ci.new_track = -1;
}
ci.seek_time = (long)ev.data+1;
break;