Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category User Interface
  • Assigned To No-one
  • Operating System Sansa AMSv2
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by idak - 2011-09-08
Last edited by kugel. - 2011-09-09

FS#12265 - Sansa Clip+ cannot display cabbiev2 wps at r30478

Sansa Clip+ cannot display cabbiev2 wps at r30478 with simulator and real.

Simulator outputs is the following.
-=-=-=-=-
Not enough skin buffer: need -65386 more.
Not enough skin buffer: need -65310 more.
Not enough skin buffer: need -65313 more.
Not enough skin buffer: need -65312 more.
SDL_WaitEvent() error
SDL_WaitEvent() error
SDL_WaitEvent() error
SDL_WaitEvent() error
-=-=-=-=-

Closed by  kugel.
2011-09-09 13:33
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

r30486

jdgordon commented on 2011-09-08 14:49

wierd, I’m looking into it but its nearly 1am here so use an older build I think.

Buschel commented on 2011-09-08 18:16

For me my own WPS (attached) does not load since r30478, cabbiev2 works fine. This is with a simulator of iPod nano 2G. The output of the simulator says: <Unable to load font 3: ‘13-arialbd.fnt’>

   BuschelBlack.wps (1.4 KiB)
Buschel commented on 2011-09-08 18:26

Attached patch fixes the issue for me.

   wps_fix_v01.patch (0.6 KiB)
kugel. commented on 2011-09-08 21:59

I think I figured the problem, I described my suspicion here: http://www.rockbox.org/irc/log-20110908#22:37:49 I’ll hopefully be able to provide a fix tomorrow.

Buschel, your problem is a different one.

jdgordon commented on 2011-09-09 00:31

Buschel: your issue is different. fonts are still loaded into the skin buffer so it means you actualy have run out of space (which is why your patch fixes it). I don’t like that #define because the needed buf size doesnt have much to do with the screen size anymore. I’m hoping to look into moveing fonts across which will fix that, but in the mean time I’d prefer a fix which adds some number*SKIN_FONT_SIZE to the SKIN_BUFFER_SIZE define instead.

Buschel commented on 2011-09-09 05:48

I’m fine with any other proper solution. I will keep my workaround until a fix has made it to svn.

kugel. commented on 2011-09-09 11:05

My suspicion was indeed the cause. The attached patch should fix it.

   0001-Buflib-Handle-not-having... (7.4 KiB)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing