Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#10549 - %mv not working properly on Wide Cabbie theme

Attached to Project: Rockbox
Opened by Michael Chicoine (mc2739) - Thursday, 20 August 2009, 14:50 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Wednesday, 09 September 2009, 07:26 GMT+2
Task Type Bugs
Category Themes
Status Closed
Assigned To Jonathan Gordon (jdgordon)
Player Type Sansa e200
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Beginning with r22418 the volume changing tag %mv does not work properly in the Wide Cabbie theme.

This theme displays a large volume graphic and volume text when the volume changes. Prior to r22418, this worked properly. Since r22418, the graphic is not displayed, but the text is displayed.

The graphic is preloaded prior to being displayed by the following line:

%xl|G|bigvolume-176x220x16.bmp|52|36|10|

The conditional to display the viewports is:

%?mv<%Vdh%Vdi|%?Fn<%?C<%Vdz%Vdd%Vde%Vdf|%Vda%Vdc>|%?C<%Vdz%Vdd%Vdg|%Vdb>>>

And the viewport definitions:

%Vl|h|0|20|-|110|-|-|-|
%?pv<%xdGa|%xdGb|%xdGc|%xdGd|%xdGe|%xdGf|%xdGg|%xdGh|%xdGi|%xdGj>
%Vl|i|0|120|-|57|-|-|-|
%acVolume: %pvdb

I have attached a screendump of the missing graphic.
   dump 090820-074436.bmp (75.7 KiB)
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Wednesday, 09 September 2009, 07:26 GMT+2
Reason for closing:  Fixed
Comment by Michael Chicoine (mc2739) - Friday, 21 August 2009, 01:10 GMT+2
This also fails in the sim with this message:

read_bmp_fd: Bitmap too large for buffer: 70560 bytes.
ERR: Failed to load image - /.rockbox/wps/widecabbie/bigvolume-176x220x16.bmp
Skin buffer usage: 18872/87120
Comment by Jonathan Gordon (jdgordon) - Friday, 21 August 2009, 01:20 GMT+2
that error says exactly what the problem is....
Comment by Michael Chicoine (mc2739) - Friday, 21 August 2009, 01:40 GMT+2
I guess this is a good theme for your "challenge: fill the skin buffer!" thread.

I got the image to load by changing the album art to 84 x 84 instead of 168 x 84. This works well with my with my album art.
Comment by Jonathan Gordon (jdgordon) - Friday, 21 August 2009, 01:47 GMT+2
wait what?! chaning the aa size shouldnt have affected anything... maybe there is another bug there
Comment by Michael Chicoine (mc2739) - Friday, 21 August 2009, 01:52 GMT+2
No, it did not fix it on the target, only the sim. Target still will not display the image.
Comment by Michael Chicoine (mc2739) - Friday, 21 August 2009, 16:21 GMT+2
I resized the bmp and got it to work.

I could not repeat the aa change making it work on the sim. I'm not sure what happened, but every attempt since the image would not show until I resized it.
Comment by Thomas Martitz (kugel.) - Saturday, 22 August 2009, 19:36 GMT+2
Working in the sim was possibly due to an outdated checkout. I just had that too, the theme broke after the commit that converted the tokens to use the skin buffer.

The theme simply exceeds the image buffer. I'm not sure if Rockbox should fail more verbosely if that happens. The problem can be solved by doing the big volume image in a more efficient way, so that it only needs 40k instead of ~90k.
Comment by Michael Chicoine (mc2739) - Saturday, 22 August 2009, 21:26 GMT+2
Why not increase the buffer slightly to where it was before r22418 so that themes are not broken.
Comment by Thomas Martitz (kugel.) - Saturday, 22 August 2009, 21:30 GMT+2
It's a single theme that is broken, and it can be fixed easily by doing it more efficiently. I see that the buffer is reduced, but I don't really see the need to increase it for everyone because of a single theme.
Comment by Michael Chicoine (mc2739) - Saturday, 22 August 2009, 21:39 GMT+2
There are actually at least 11 themes for the e200 (the original and others based on the original) that are now broken. That does not include themes that may be broken due to the smaller buffer that have not been identified yet. My thought is that an increase in the buffer size is better than breaking existing themes, especially since, if I remember correctly, the changes caused a binsize reduction.
Comment by Michael Chicoine (mc2739) - Monday, 24 August 2009, 04:36 GMT+2
Widecabbie now works again as of r22474

e200 (and e200v2) System->Rockbox Info: Skin RAM usage: 87.4KB / 93.0KB
Comment by Michael Chicoine (mc2739) - Monday, 24 August 2009, 05:06 GMT+2
Ram usage numbers from e200 sim:

Skin buffer usage: 89432/95312
System->Rockbox Info: Skin RAM usage: 87.3KB / 93.0KB

Loading...