diff options
author | Michael Sevakis <jethead71@rockbox.org> | 2013-07-14 07:59:39 -0400 |
---|---|---|
committer | Michael Sevakis <jethead71@rockbox.org> | 2014-03-10 04:12:30 +0100 |
commit | 31b712286721dd606940c7b557d03e3f714b9604 (patch) | |
tree | 8c4a4cc32e9000ea721ebb23aa3c0129ca97bf53 /apps/tagtree.c | |
parent | dda54b85daa83b7803b4fb189ab45859f96ff3f9 (diff) | |
download | rockbox-31b712286721dd606940c7b557d03e3f714b9604.tar.gz rockbox-31b712286721dd606940c7b557d03e3f714b9604.zip |
Implement time-based resume and playback start.
This complements offset-based resume and playback start funcionality.
The implementation is global on both HWCODEC and SWCODEC.
Basically, if either the specified elapsed or offset are non-zero,
it indicates a mid-track resume.
To resume by time only, set elapsed to nonzero and offset to zero.
To resume by offset only, set offset to nonzero and elapsed to zero.
Which one the codec uses and which has priority is up to the codec;
however, using an elapsed time covers more cases:
* Codecs not able to use an offset such as VGM or other atomic
formats
* Starting playback at a nonzero elapsed time from a source that
contains no offset, such as a cuesheet
The change re-versions pretty much everything from tagcache to nvram.
Change-Id: Ic7aebb24e99a03ae99585c5e236eba960d163f38
Reviewed-on: http://gerrit.rockbox.org/516
Reviewed-by: Michael Sevakis <jethead71@rockbox.org>
Tested: Michael Sevakis <jethead71@rockbox.org>
Diffstat (limited to 'apps/tagtree.c')
-rw-r--r-- | apps/tagtree.c | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/apps/tagtree.c b/apps/tagtree.c index 9eef38b5e6..ff364ec5e4 100644 --- a/apps/tagtree.c +++ b/apps/tagtree.c | |||
@@ -330,6 +330,7 @@ static int get_tag(int *tag) | |||
330 | {"playcount", tag_playcount}, | 330 | {"playcount", tag_playcount}, |
331 | {"rating", tag_rating}, | 331 | {"rating", tag_rating}, |
332 | {"lastplayed", tag_lastplayed}, | 332 | {"lastplayed", tag_lastplayed}, |
333 | {"lastelapsed", tag_lastelapsed}, | ||
333 | {"lastoffset", tag_lastoffset}, | 334 | {"lastoffset", tag_lastoffset}, |
334 | {"commitid", tag_commitid}, | 335 | {"commitid", tag_commitid}, |
335 | {"entryage", tag_virt_entryage}, | 336 | {"entryage", tag_virt_entryage}, |
@@ -841,8 +842,16 @@ static void tagtree_buffer_event(void *data) | |||
841 | #if CONFIG_CODEC == SWCODEC | 842 | #if CONFIG_CODEC == SWCODEC |
842 | if (autoresume) | 843 | if (autoresume) |
843 | { | 844 | { |
844 | /* Load current file resume offset if not already defined (by | 845 | /* Load current file resume info if not already defined (by |
845 | another resume mechanism) */ | 846 | another resume mechanism) */ |
847 | if (id3->elapsed == 0) | ||
848 | { | ||
849 | id3->elapsed = tagcache_get_numeric(&tcs, tag_lastelapsed); | ||
850 | |||
851 | logf("tagtree_buffer_event: Set elapsed for %s to %lX\n", | ||
852 | str_or_empty(id3->title), id3->elapsed); | ||
853 | } | ||
854 | |||
846 | if (id3->offset == 0) | 855 | if (id3->offset == 0) |
847 | { | 856 | { |
848 | id3->offset = tagcache_get_numeric(&tcs, tag_lastoffset); | 857 | id3->offset = tagcache_get_numeric(&tcs, tag_lastoffset); |
@@ -940,12 +949,13 @@ static void tagtree_track_finish_event(void *data) | |||
940 | #if CONFIG_CODEC == SWCODEC | 949 | #if CONFIG_CODEC == SWCODEC |
941 | if (autoresume) | 950 | if (autoresume) |
942 | { | 951 | { |
952 | unsigned long elapsed = auto_skip ? 0 : id3->elapsed; | ||
943 | unsigned long offset = auto_skip ? 0 : id3->offset; | 953 | unsigned long offset = auto_skip ? 0 : id3->offset; |
944 | 954 | tagcache_update_numeric(tagcache_idx, tag_lastelapsed, elapsed); | |
945 | tagcache_update_numeric(tagcache_idx, tag_lastoffset, offset); | 955 | tagcache_update_numeric(tagcache_idx, tag_lastoffset, offset); |
946 | 956 | ||
947 | logf("tagtree_track_finish_event: Save offset for %s: %lX", | 957 | logf("tagtree_track_finish_event: Save resume for %s: %lX %lX", |
948 | str_or_empty(id3->title), offset); | 958 | str_or_empty(id3->title), elapsed, offset); |
949 | } | 959 | } |
950 | #endif | 960 | #endif |
951 | } | 961 | } |
@@ -2034,7 +2044,7 @@ static int tagtree_play_folder(struct tree_context* c) | |||
2034 | c->selected_item = 0; | 2044 | c->selected_item = 0; |
2035 | gui_synclist_select_item(&tree_lists, c->selected_item); | 2045 | gui_synclist_select_item(&tree_lists, c->selected_item); |
2036 | 2046 | ||
2037 | playlist_start(c->selected_item,0); | 2047 | playlist_start(c->selected_item, 0, 0); |
2038 | playlist_get_current()->num_inserted_tracks = 0; /* make warn on playlist erase work */ | 2048 | playlist_get_current()->num_inserted_tracks = 0; /* make warn on playlist erase work */ |
2039 | return 0; | 2049 | return 0; |
2040 | } | 2050 | } |