This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#8828 - Image check of wps doesn't lead to hard-coded wps screen
Attached to Project:
Rockbox
Opened by Thomas Martitz (kugel.) - Saturday, 29 March 2008, 22:03 GMT+2
Last edited by Nicolas Pennequin (nicolas_p) - Tuesday, 01 April 2008, 19:47 GMT+2
Opened by Thomas Martitz (kugel.) - Saturday, 29 March 2008, 22:03 GMT+2
Last edited by Nicolas Pennequin (nicolas_p) - Tuesday, 01 April 2008, 19:47 GMT+2
|
DetailsNormally, if a syntax error in a .wps occurs, rockbox will reset the wps and apply the very default one (not the wps of cabbiev2, the hard-coded one).
Lately, there was a commit which checked if all images, which the wps want to load can be loaded properly. If the check fails, the wps is rejected. This is not the case. Just all images don't get loaded, everything else will apply (i.e. the progress bar image will not used, but the progress bar dimensions and position are still apllied). To reproduce: Download a wps, which has some flawed images, i.e. this one: http://www.rockbox.org/twiki/bin/view/Main/WpsSansaE200#Razor (razor). Load the wps and see yourself. IMO it should reject the entire wps and show the hard-coded wps, exactly what happens if syntax errors occur. Tested on e200 (sim and target, svn revision from 29th of March), but it possibly applies for all targets. I talked about that with linuxstb on IRC (log of 29th of march; 20:22:15 CET) allready, and he says that it should be fixed. |
This task depends upon
Closed by Nicolas Pennequin (nicolas_p)
Tuesday, 01 April 2008, 19:47 GMT+2
Reason for closing: Fixed
Additional comments about closing: r16914
Tuesday, 01 April 2008, 19:47 GMT+2
Reason for closing: Fixed
Additional comments about closing: r16914