Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8763 - Add posibility of 10 more predefined images in WPS definition

Attached to Project: Rockbox
Opened by Henrik Johnson (hpj) - Thursday, 20 March 2008, 06:52 GMT+2
Last edited by Dave Chapman (linuxstb) - Monday, 24 March 2008, 00:46 GMT+2
Task Type Patches
Category WPS
Status Closed
Assigned To No-one
Player type All players
Severity Low
Priority Normal
Reported Version current build
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Very simple patch to add support for 10 more predefined images. Apart from a-zA-Z as ID:s you can with this also add 0-9. Seems like a fairly low impact change and since at least for me I run into the image index limit all the time (I really like having 11 images for the rating).
This task depends upon

Closed by  Dave Chapman (linuxstb)
Monday, 24 March 2008, 00:46 GMT+2
Reason for closing:  Rejected
Additional comments about closing:  The "bitmap strips" feature is now implemented, which in theory allows up to 2704 bitmaps in a WPS.
Comment by Henrik Johnson (hpj) - Thursday, 20 March 2008, 06:53 GMT+2
Huh, somehow the patch didn't stick. Here it is anyway.
   extra10image.diff (0.7 KiB)
Comment by Dave Chapman (linuxstb) - Thursday, 20 March 2008, 09:52 GMT+2
IMO, (and this idea has been discussed in IRC a few times, but has got no further) a better solution to the problem of the image limitation would be to add support for "icon bitmap strips". i.e. all images for the same conditional (volume, battery, codec type etc) would be in a single bitmap (arranged vertically), and then Rockbox would simply display the appropriate bitmap from that strip, depending on the value of the conditional.

This would obviously greatly reduce the number of bitmaps needed in a WPS.

Loading...