Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11103 - MPC resume doesn't always work

Attached to Project: Rockbox
Opened by Magnus Holmgren (learman) - Sunday, 14 March 2010, 10:52 GMT+2
Last edited by Andree Buschmann (Buschel) - Sunday, 14 March 2010, 14:33 GMT+2
Task Type Bugs
Category Codecs
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Resuming a MPC track doesn't always work; sometimes it start at 0:00. This seem to happens because the MPC codec use id3->offset field as a sample offset, while the Rockbox core use it as file offset. If the resume offset is bigger than the file size, it is reset to zero by the playback code. The mpc decoder library only has a private low-level function for seeking to a particular file offset, so it isn't a trivial fix.
This task depends upon

Closed by  Andree Buschmann (Buschel)
Sunday, 14 March 2010, 14:33 GMT+2
Reason for closing:  Fixed
Additional comments about closing:  Fixed with r25164 (with additional change to keep precision of time position over save/resume).
Comment by Andree Buschmann (Buschel) - Sunday, 14 March 2010, 14:24 GMT+2
Good point! This failure happened since ages...

The following patch corrects the usage of rockbox's "offset" when resuming playback and when saving the current position for persistency.
   mpc_resume_v02.patch (2 KiB) 
 apps/codecs/mpc.c |   19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

Loading...