This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#11526 - Use of %Vb/f (viewport foreground and background) break WPS or RWPS on X5
Attached to Project:
Rockbox
Opened by Marianne Arnold (pixelma) - Monday, 09 August 2010, 11:53 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Tuesday, 10 August 2010, 16:15 GMT+2
Opened by Marianne Arnold (pixelma) - Monday, 09 August 2010, 11:53 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Tuesday, 10 August 2010, 16:15 GMT+2
|
DetailsThe X5 has a colour display and a greyscale remote, hence it was possible to set a viewport foreground and background (as RGB on main screen and as one of the greyshades for the remote) - this is broken in current SVN builds.
A WPS for the main screen which uses %Vb(FF00FF) is rejected by the parser and you get the default WPS, %Vb(3) which would be wrong for colour is rejected as well. The theme site also only lists a few themes - there should be as many as e.g. the 20GB H10 ones but there are less than the 160x128 greyscale targets have. Themes/WPSs listed are all without %Vb/f. An RWPS which uses %Vf(0) is rejected as well. Unfortunately the cabbiev2.rwps uses %Vb/f in one place and doesn't show up (can't remember completely why it was done this way but that shouldn't matter here). The same .wps works fine on the M3 as main wps and also on the M5 as RWPS... This last fact leads me to assume that it could have something to do with the different formats the value of %Vb/f needs (RGB and greyshade). The problem is visible in a simulator and was also present in revision 27663, one before "r27664: make grey targets reject viewport colours if they are more than one letter (i.e %Vb(333333) )". It was also confirmed or reported by Temik on IRC who saw it on target. |
This task depends upon
Closed by Jonathan Gordon (jdgordon)
Tuesday, 10 August 2010, 16:15 GMT+2
Reason for closing: Fixed
Additional comments about closing: in r27768.
Tuesday, 10 August 2010, 16:15 GMT+2
Reason for closing: Fixed
Additional comments about closing: in r27768.