FS#8828 - Image check of wps doesn't lead to hard-coded wps screen
Opened by Thomas Martitz (kugel.) - Saturday, 29 March 2008, 21:03 GMT
Last edited by Nicolas Pennequin (nicolas_p) - Tuesday, 01 April 2008, 17:47 GMT
Normally, 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).
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.
Tuesday, 01 April 2008, 17:47 GMT
Reason for closing: Fixed
Additional comments about closing: r16914