From 4844142e161f24102bbe45f1f6207f3468649e78 Mon Sep 17 00:00:00 2001 From: Michael Hohmuth Date: Tue, 8 Feb 2011 20:31:27 +0000 Subject: Do not update resume information and do not log statistics when a track is interrupted within the first 15 seconds. Regard a rewind to 0:00 as a track restart (updating resume position / playback statistics before the rewind and starting the 15 s delay). This allows skipping forward across an unplayed track without changing its resume offset. Also, it is possible to skip backward to the previous track after rewinding to the current track to 0:00 (pressing Left twice) without losing the current track's resume position. Initially contributed by Dave Slusher Caveats: * Works only for SWCODEC * Skipping forward without altering the resume position does not work when skip to outro has been turned on. * The 15-second window in which the resume offset will not be updated should start at the initial resume position, not at 0:00. This would allow skipping over partially played tracks without altering the resume position. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@29250 a1c6a512-1295-4272-9138-f99709370657 --- apps/tagtree.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'apps/tagtree.c') diff --git a/apps/tagtree.c b/apps/tagtree.c index fa88a5941b..ecc9f44d4d 100644 --- a/apps/tagtree.c +++ b/apps/tagtree.c @@ -722,10 +722,15 @@ static void tagtree_track_finish_event(void *data) } tagcache_idx--; - /* Don't process unplayed tracks. */ - if (id3->elapsed == 0) + /* Don't process unplayed tracks, or tracks interrupted within the + first 15 seconds. */ + if (id3->elapsed == 0 +#if CONFIG_CODEC == SWCODEC /* HWCODEC doesn't have automatic_skip */ + || (id3->elapsed < 15 * 1000 && !automatic_skip) +#endif + ) { - logf("not logging unplayed track"); + logf("not logging unplayed or skipped track"); return; } -- cgit v1.2.3