- Status Closed
- Percent Complete
- Task Type Bugs
- Category User Interface → Themes
- Assigned To No-one
- Operating System iPod 5G
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#12320 - need substitute to /.rockbox/skin_buffer_size.txt
Since version r30718 “Revert r30599 “Check for the magic file “/.rockbox/skin_buffer_size.txt” my WPS is not loading again. With the last changes to buflib my WPS didn’t work until I’ve used “/.rockbox/skin_buffer_size.txt” to load my WPS that is 47.2KB in size (”system > rockbox info > skin buffer usage”).
Is there a way to fit my WPS on the buffer? (My theme is BigInfoAA on the Themes page)
At the moment, I switch it back to r30717.
Closed by jdgordon
2011-11-14 11:31
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
2011-11-14 11:31
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
proper fix should be in svn in the next
day or two
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Can you try this patch? It optimizes one often allocated struct for size.
That saves barely 2K on e200 cabbie (run the two attached .log files through lib/skin_parser/skin_buffer_debug.pl). i.e a completely pointless patch. The *only* correct fix is to figure out how to get that struct onto buflib, all other patches are a waste of time.
edit: the next biggest allocation is apps/gui/skin_engine/skin_parser.c:1851 but even that is barely 2k (on cabbie), and this is an easier one to convert to buflib but even that is pointless
I've tried it and it didn't work for me. :'(
How do you use skin_buffer_debug.pl, or how do you generate the logs?
skin_buffer_debug.pl file.log | sort -k2 -nr
enable the debug line in libs/skin_engine/skin_buffer.h
Okay, making the structs smaller obviously doesn't help since the skin buffer takes their size into account. MAX_WPS_TOKEN can be increased according to the space saved by the smaller structs.
This does bump MAX_WPS_TOKEN a bit, for the space changed by decreasing the struct sizes. Makes BigInfoAA fit on the 64bit sim
How exactly is that better than the change you reverted? It is still an arbitrary limit. OK, BigInfoAA fits but some other skin wont. The only sane thing to do is put skin_element on buflib or make the buffer size configurable
It's a fix to a regression introduced by the skin-engine-to-buflib changes. Without extra cost, and without introducing questionable new features.
This afternoon I'll give it a try.
Thanks.
It works for me. Is this a definitive solution, or is there anything I could try/do to help find a better solution?
The commit is a stop gap solution but not a real fix. This task shouldnt have been closed.