rockbox/lib/skin_parser/skin_parser.c, branch duke3d

Limit more variables to file scope

2015-01-11T20:40:51+00:00

Change-Id: I30219d626316776eb73b4205d63376fa3dbc6361

Fix yellow

2015-01-05T18:09:33+00:00

Change-Id: I8685198c208b5324b09b5ad59f7379502e9ed977

Make thirty functions static to reduce binary size

2015-01-05T17:44:36+00:00

If any of those functions should be (unused) API functions,
they can easily be turned back once really needed.

Detected using a new cppcheck check that
uses the internal symbol database to catch
functions that are only used in the current file.

Change-Id: Ic2b1e5b8020b76397f11cefc4e205f3b7ac1f184

more error handling for checkwps

2013-02-27T10:15:57+00:00

Change-Id: I03055d045c0a8e0e63e17b290cc71c54a8dc3634

checkwps: show a helpful error if the parser callback errors out

2013-02-26T10:18:16+00:00

Change-Id: Ie3e35292ba8d74f0ff3d1bb3483a5e83aae0e6b6

skin_engine: Support percentages for viewport positioning

2012-07-29T02:24:38+00:00

%V(0,50%,75%,50%,-) - make a viewport at x=0, y=half the lcd height,
75% lcd width and the remaining height (the other half) of the lcd.

Change-Id: If26ccb65e8dc52c9225f3fd6d7b222d770add0f0
Reviewed-on: http://gerrit.rockbox.org/184
Reviewed-by: Thomas Martitz 
Tested-by: Thomas Martitz 
Reviewed-by: Jonathan Gordon

skin_engine: Fix a obscure parser bug

2012-07-05T12:41:10+00:00

Tag params where a [] group is followed (but not immediatly) by
a * will not parse correctly. e.g [si]iii|s* will attempt to find
an 'i' after the second s instead of looping s's

Change-Id: I3982f726b6539818f8332334b263b673259f98ef

skin_engine: rework the parser to be closer to the langauge grammar.

2012-04-21T14:24:16+00:00

The parser was unconditionally scanning things which it thought
were conditional/enum lists (or tag arg lists) when they couldn't
possibly be (i.e < inside a param which should be valid).

This change fixes it (i.e %?and(%if(%pv, <, -50), %if(%mp, >i, 1))
is perfectly valid now.

This *may* break your exsiting skins if you were using %if with < or >

Change-Id: Ia24dbdf0b11fc7d8a735c1111d648c3bebd68ac6

Revert "skin_engine: rework the parser to be closer to the langauge grammar."

2012-04-17T15:07:43+00:00

This reverts commit ec8b21eef8b2fe1bd02f335dbc0dfbf05c2deff2 which was pushed by accident.

Change-Id: I1aaedf6876d0448a100dc582b79f1293d021bac1
Reviewed-on: http://gerrit.rockbox.org/216
Reviewed-by: Thomas Martitz

skin_engine: rework the parser to be closer to the langauge grammar.

2012-04-17T15:05:20+00:00

The parser was unconditionally scanning things which it thought
were conditional/enum lists (or tag arg lists) when they couldn't
possibly be (i.e < inside a param which should be valid).

This change fixes it (i.e %?and(%if(%pv, <, -50), %if(%mp, > 1))
is perfectly valid now.

This *may* break your exsiting skins if you were using %if with < or >

Change-Id: Ibcb42bc6bb78908f79de024b61276b91b1ce02a0
Reviewed-on: http://gerrit.rockbox.org/214
Reviewed-by: Thomas Martitz