- Status Closed
- Percent Complete
- Task Type Patches
- Category User Interface
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Release 3.4
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#11265 - Beginning of the theme editor parser
Laid some of the groundwork for a generic WPS parser for JdGordon’s skinenginebreak branch
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
Updated version with tag table
Working on parser
Working on parse_tag
the malloc line in skin_alloc_string(int length) looks like it alloca double the needed bytes?
other than that, I havnt looked at it enough to comment…
Good catch, I had the + 1 (to accomodate the \0 at the end of the string) in the wrong place. Here’s a new patch (hopefully I’ll have SVN access soon) that SHOULD parse any WPS document in your new format successfully, as long as the tags have been added to the tag table. I assumed that we’d use a conditional of the form
%?xx(option1,option2,option3)
since none was specified in your wiki document, just let me know if you’d rather some other format.