Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#12806 - Checkwps fails, but skin works on sim/target

Attached to Project: Rockbox
Opened by Hayden Pearce (St.) - Monday, 21 January 2013, 01:34 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Tuesday, 22 January 2013, 21:44 GMT+2
Task Type Bugs
Category User Interface
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

The parser is barfing on syntactically correct statements, for example:

Checking /home/saint/Desktop/iLike Classic/Current/iLike Classic/wps/iLike.wps...
WPS parsing failure
Error on line 67.
%?bp<%?bc<%t(1.0)%xd(Ea);%t(1.0)%xd(Eb)|%xd(Ec)>|%bl(1,1,21,10,image,B,backdrop,
^ 'Expected argument separator' Here
saint@saint-desktop:~/Development/Rockbox/development/rockbox/build$

"%?bp<%?bc<|>|>" parses correctly.

"%?bp<%?bc<%t(1.0)%xd(Ea);%t(1.0)%xd(Eb)|%xd(Ec)>|%bl(1,1,21,10,image,B,backdrop,C)>" is rejected by checkwps and isn't displayed by either the sim or device.

"%?bp<%?bc<%t(1.0)%xd(Aa);%t(1.0)%xd(Ab)|%xd(Ac)>|%?bl<%xd(Ad)|%xd(Ad)|%xd(Ae)|%xd(Af)|%xd(Ag)|%xd(Ah)|%xd(Ai)|%xd(Aj)|%xd(Ak)|%xd(Al)|%xd(Am)|%xd(An)|%xd(Ao)|%xd(Ap)|%xd(Aq)|%xd(Ar)|%xd(As)|%xd(At)|%xd(Au)|%xd(Av)|%xd(Aw)|%xd(Ax)|%xd(Ay)|%xd(Az)>>" is rejected by checkwps but is displayed correctly in the sim and on device.


[Saint]
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Tuesday, 22 January 2013, 21:44 GMT+2
Reason for closing:  Not a Bug
Additional comments about closing:  problem user not checkwps
Comment by Hayden Pearce (St.) - Monday, 21 January 2013, 01:45 GMT+2
full theme .zip attatched:
   iLike Classic.zip (70 KiB)
Comment by Hayden Pearce (St.) - Monday, 21 January 2013, 01:50 GMT+2
A .zip of the same theme before changing the syntax around showing syntax that works in the sim and on device but fails checkwps:
   iLike Classic -old.zip (71.6 KiB)
Comment by Michael Sevakis (MikeS) - Tuesday, 22 January 2013, 17:05 GMT+2
Jd, why the removal of critical technical terminology from the title?
Comment by Jonathan Gordon (jdgordon) - Tuesday, 22 January 2013, 21:44 GMT+2
:)

Loading...