Rockbox

Tasklist

FS#10576 - move sub/lines to skin buffer

Attached to Project: Rockbox
Opened by Jonathan Gordon (jdgordon) - Monday, 31 August 2009, 00:27 GMT
Last edited by Jonathan Gordon (jdgordon) - Wednesday, 02 September 2009, 03:59 GMT
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

this patch seems to "work", but the debug output just looks very wrong.. also it crashes the sim if --debugwps is enabled...

(gdb) p *v->lines->next->sublines
$8 = {first_token_idx = 4, last_token_idx = 3, line_type = 0 '\0',
time_mult = 0 '\0', next = 0x0}

^ see anything wrong there? how can last be before first?!... I'm hoping another set of eyes will have an idea where its screwed up
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Wednesday, 02 September 2009, 03:59 GMT
Reason for closing:  Accepted
Comment by Jonathan Gordon (jdgordon) - Monday, 31 August 2009, 15:41 GMT
forgot to update this before going to bed...
its slightly better but apparently crashes the ipod....

the wierd stuff with the token numbers above is because for blank new lines, the \n isnt stored as a token, so special handling is needed (not added yet)
Comment by Jonathan Gordon (jdgordon) - Tuesday, 01 September 2009, 01:08 GMT
updated to add some special handling for empty lines
Comment by Jonathan Gordon (jdgordon) - Tuesday, 01 September 2009, 07:00 GMT
probably a lost cause but anyway.... somehow curr_line is getting set to 0x1 or some other similarly stupid value during parseing... I havnt figured out where or why or how though :( an internets to the person who figures it out!
Comment by Jonathan Gordon (jdgordon) - Tuesday, 01 September 2009, 07:52 GMT
All glory to funman for figuring it out :D
apparently there is still an isue to figure out though :/
Comment by Jonathan Gordon (jdgordon) - Wednesday, 02 September 2009, 00:41 GMT
I'm not memset()ing the newly alloc buffers to 0's which unfortunatly hides the real issue, but makes it work...

Loading...