• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Plugins
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.0
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by kugel. - 2008-12-12
Last edited by rasher - 2008-12-12

FS#9627 - Pictureflow: 3 bugfixes

This patch fixes all remaining bugs I know about :

 FS#9606  - by getting the whole metadata, not only album
 FS#9135  - by explicitly removing existing cache before creating new one (i.e. that what the user had to do manually)
 FS#8347  - by not resetting the selection variable upon re-entering (now works as expected, i.e. not by dirty resetting tracklist)

Closed by  rasher
2008-12-12 21:29
Reason for closing:  Accepted
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

Committed in revision 19408.

Your patch has a lot of seemingly unrelated scrollwheel-related changes. Please post a cleaner patch.

Then add this as a 4th bug fix.

The scrollwheel define in pf is homebrewed and doesn’t use the existing HAVE_SCROLLWHEEL. Imagine the fuze gets supported, pf will be broken, even though you’d think everything is fine if you defined HAVE_SCROLLWHEEL.

Okok, take this one :)


Available keyboard shortcuts


Task Details

Task Editing