Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#11477 - Add support for decimals in the skin_parser lib

Attached to Project: Rockbox
Opened by Jonathan Gordon (jdgordon) - Wednesday, 14 July 2010, 13:21 GMT+2
Last edited by Jonathan Gordon (jdgordon) - Thursday, 15 July 2010, 11:08 GMT+2
Task Type Patches
Category Themes
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Release 3.6
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

This adds the 'd' option to the parser which will parse a decimal value with one decimal digit precision and store it in the data.number value as (whole*10)+part (so 12.3 -> 123).

This compiles on the theme editor but I dont know where to add proper support (includign the changed tags).
   decimal.patch (16.2 KiB)
 b/lib/skin_parser/skin_debug.c                 |   12 +++++-
 b/lib/skin_parser/skin_parser.c                |   39 +++++++++++++++++++--
 b/lib/skin_parser/skin_parser.h                |    6 ++-
 b/lib/skin_parser/tag_table.c                  |   10 ++---
 b/lib/skin_parser/tag_table.h                  |    3 +
 b/utils/themeeditor/graphics/rbprogressbar.cpp |    8 ++--
 b/utils/themeeditor/graphics/rbviewport.cpp    |   10 ++---
 b/utils/themeeditor/gui/devicestate.cpp        |    2 -
 b/utils/themeeditor/models/parsetreemodel.cpp  |    4 +-
 b/utils/themeeditor/models/parsetreenode.cpp   |   46 ++++++++++++-------------
 10 files changed, 93 insertions(+), 47 deletions(-)

This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Thursday, 15 July 2010, 11:08 GMT+2
Reason for closing:  Accepted
Comment by Jonathan Gordon (jdgordon) - Wednesday, 14 July 2010, 14:01 GMT+2
woops, try this one.
   decimal.patch (16.2 KiB)
 b/lib/skin_parser/skin_debug.c                 |   12 +++++-
 b/lib/skin_parser/skin_parser.c                |   39 +++++++++++++++++++--
 b/lib/skin_parser/skin_parser.h                |    6 ++-
 b/lib/skin_parser/tag_table.c                  |   10 ++---
 b/lib/skin_parser/tag_table.h                  |    3 +
 b/utils/themeeditor/graphics/rbprogressbar.cpp |    8 ++--
 b/utils/themeeditor/graphics/rbviewport.cpp    |   10 ++---
 b/utils/themeeditor/gui/devicestate.cpp        |    2 -
 b/utils/themeeditor/models/parsetreemodel.cpp  |    4 +-
 b/utils/themeeditor/models/parsetreenode.cpp   |   46 ++++++++++++-------------
 10 files changed, 93 insertions(+), 47 deletions(-)

Loading...