Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category User Interface
  • Assigned To
    MikeS
  • Operating System SW-codec
  • Severity Low
  • Priority Low
  • Reported Version Release 3.8.1
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by MikeS - 2011-05-23
Last edited by MikeS - 2011-05-25

FS#12127 - Add a track ready event to be sure album art and cuesheets are properly recognized

A problem was mentioned in rockbox-dev where album art might not get picked up initially when starting playback (http://www.rockbox.org/mail/archive/rockbox-dev-archive-2011-05/0000.shtml). I can see how the problem can happen and a way to rememdy it, at least for now. This adds an event that is fired when the remaining handles are finished that the UI has to care about, iff the track was only partially loaded at the time the new track event was fired.

Ok, so maybe it’ll stay or maybe it won’t. Some other strategies are possible in the way things work underneath that could make such an event unneeded but the changes would be somewhat deep.

Closed by  MikeS
2011-05-25 08:39
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Reporter says this change takes care of the issue. Discuss further in rockbox-dev where the report was first posted if any more discussion is required.

Made a quick test, for a known problem case, and it worked fine with the patch applied.

MikeS commented on 2011-05-25 03:03

What is the particular problem case? I have quite a bit of random stuff with embedded AA and whatnot and it never seems to show itself. Why it even works at all the way it is in SVN is a bit strange, stranger than it not working and yet…

Ok, I’ll think about commit tonight even if it’s somewhat of a workabout for the moment since I’ve got other stuff in the pipelines I’m not ready to mess up.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing