Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#10617 - %Cl and %C conditional viewport oddity

Attached to Project: Rockbox
Opened by Seheon Ryu (cpu98) - Friday, 25 September 2009, 17:08 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Monday, 05 October 2009, 12:26 GMT+2
Task Type Bugs
Category Themes
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

-------------------------------
%Cl|0|38|145|145|
%?C<%C%Vda|%Vdb>

Viewport b is used even if AA is not present.

-------------------------------
%Cl|0|38|145|145|

%?C<%C%Vda|%Vdb>

Viewport b is used when AA is not present.(expected behavior)

Really bugged me......
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Monday, 05 October 2009, 12:26 GMT+2
Reason for closing:  Fixed
Additional comments about closing:  shuold be fixed in r22944.
Comment by Seheon Ryu (cpu98) - Friday, 25 September 2009, 17:08 GMT+2
edit: Viewport b is used even if AA is not present. -> Viewport b is used even if AA is present.
Comment by Seheon Ryu (cpu98) - Friday, 25 September 2009, 17:12 GMT+2
%Cl|0|38|145|145|
%?C<%Vda|%Vdb>
Behaves as if it is given
%Cl|0|38|145|145|
%C
%V(viewport b)
Comment by Jonathan Gordon (jdgordon) - Friday, 25 September 2009, 18:34 GMT+2
whats the build number you are using? (menu > system > rockbox info: )
Comment by Seheon Ryu (cpu98) - Saturday, 26 September 2009, 12:11 GMT+2
Quite recent revision. 22827.
Need wps file for testing?
Comment by Teruaki Kawashima (teru) - Sunday, 27 September 2009, 15:05 GMT+2
i'm not sure if this is related to the bug.
skin_parser.c:1138: if ((wps_data->num_tokens > 1) && (prev->type == WPS_TOKEN_CONDITIONAL))
Shouldn't (wps_data->num_tokens > 1) be (wps_data->num_tokens >= 1) ?
Comment by Jonathan Gordon (jdgordon) - Sunday, 27 September 2009, 18:11 GMT+2
ah crud... yes... I forgot the Cl isnt stored as a token... I'll fix that this arvo

Loading...