<feed xmlns='http://www.w3.org/2005/Atom'>
<title>rockbox/lib/skin_parser/skin_scan.c, branch master</title>
<subtitle>My Rockbox tree</subtitle>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/'/>
<entry>
<title>Fix yellow</title>
<updated>2015-01-05T18:09:33+00:00</updated>
<author>
<name>Thomas Jarosch</name>
<email>tomj@simonv.com</email>
</author>
<published>2015-01-05T18:09:33+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=f91434cc7b4204ca47c86090928cdd454dd52606'/>
<id>f91434cc7b4204ca47c86090928cdd454dd52606</id>
<content type='text'>
Change-Id: I8685198c208b5324b09b5ad59f7379502e9ed977
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I8685198c208b5324b09b5ad59f7379502e9ed977
</pre>
</div>
</content>
</entry>
<entry>
<title>Make thirty functions static to reduce binary size</title>
<updated>2015-01-05T17:44:36+00:00</updated>
<author>
<name>Thomas Jarosch</name>
<email>tomj@simonv.com</email>
</author>
<published>2015-01-05T17:44:36+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=fdd4aef34003587d7fd9ed754dd35ce901b639bf'/>
<id>fdd4aef34003587d7fd9ed754dd35ce901b639bf</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>skin_engine: rework the parser to be closer to the langauge grammar.</title>
<updated>2012-04-21T14:24:16+00:00</updated>
<author>
<name>Jonathan Gordon</name>
<email>rockbox@jdgordon.info</email>
</author>
<published>2012-04-21T13:34:42+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=2d3c43dffeb048297313cd9333731d47d66010a0'/>
<id>2d3c43dffeb048297313cd9333731d47d66010a0</id>
<content type='text'>
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 &lt; inside a param which should be valid).

This change fixes it (i.e %?and(%if(%pv, &lt;, -50), %if(%mp, &gt;i, 1))
is perfectly valid now.

This *may* break your exsiting skins if you were using %if with &lt; or &gt;

Change-Id: Ia24dbdf0b11fc7d8a735c1111d648c3bebd68ac6
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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 &lt; inside a param which should be valid).

This change fixes it (i.e %?and(%if(%pv, &lt;, -50), %if(%mp, &gt;i, 1))
is perfectly valid now.

This *may* break your exsiting skins if you were using %if with &lt; or &gt;

Change-Id: Ia24dbdf0b11fc7d8a735c1111d648c3bebd68ac6
</pre>
</div>
</content>
</entry>
<entry>
<title>Revert "skin_engine: rework the parser to be closer to the langauge grammar."</title>
<updated>2012-04-17T15:07:43+00:00</updated>
<author>
<name>Thomas Martitz</name>
<email>kugel@rockbox.org</email>
</author>
<published>2012-04-17T15:06:51+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=728db2150277206ae82d6f5ed82b86cf779ee653'/>
<id>728db2150277206ae82d6f5ed82b86cf779ee653</id>
<content type='text'>
This reverts commit ec8b21eef8b2fe1bd02f335dbc0dfbf05c2deff2 which was pushed by accident.

Change-Id: I1aaedf6876d0448a100dc582b79f1293d021bac1
Reviewed-on: http://gerrit.rockbox.org/216
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This reverts commit ec8b21eef8b2fe1bd02f335dbc0dfbf05c2deff2 which was pushed by accident.

Change-Id: I1aaedf6876d0448a100dc582b79f1293d021bac1
Reviewed-on: http://gerrit.rockbox.org/216
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>skin_engine: rework the parser to be closer to the langauge grammar.</title>
<updated>2012-04-17T15:05:20+00:00</updated>
<author>
<name>Jonathan Gordon</name>
<email>rockbox@jdgordon.info</email>
</author>
<published>2012-04-16T05:42:24+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=e43b856ed0f2fa3cb03a1335c9dc311b572e88e2'/>
<id>e43b856ed0f2fa3cb03a1335c9dc311b572e88e2</id>
<content type='text'>
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 &lt; inside a param which should be valid).

This change fixes it (i.e %?and(%if(%pv, &lt;, -50), %if(%mp, &gt; 1))
is perfectly valid now.

This *may* break your exsiting skins if you were using %if with &lt; or &gt;

Change-Id: Ibcb42bc6bb78908f79de024b61276b91b1ce02a0
Reviewed-on: http://gerrit.rockbox.org/214
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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 &lt; inside a param which should be valid).

This change fixes it (i.e %?and(%if(%pv, &lt;, -50), %if(%mp, &gt; 1))
is perfectly valid now.

This *may* break your exsiting skins if you were using %if with &lt; or &gt;

Change-Id: Ibcb42bc6bb78908f79de024b61276b91b1ce02a0
Reviewed-on: http://gerrit.rockbox.org/214
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Correct spelling of 'seperate' in the skin parsing code - FS#11724 by Alexander Levin</title>
<updated>2010-11-05T09:51:19+00:00</updated>
<author>
<name>Bertrik Sikken</name>
<email>bertrik@sikken.nl</email>
</author>
<published>2010-11-05T09:51:19+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=fffbdcc9960b6a2d8bf50f3c67674278780ad00f'/>
<id>fffbdcc9960b6a2d8bf50f3c67674278780ad00f</id>
<content type='text'>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@28485 a1c6a512-1295-4272-9138-f99709370657
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@28485 a1c6a512-1295-4272-9138-f99709370657
</pre>
</div>
</content>
</entry>
<entry>
<title>skinparser lib: more const correctness</title>
<updated>2010-07-31T16:25:41+00:00</updated>
<author>
<name>Nils Wallménius</name>
<email>nils@rockbox.org</email>
</author>
<published>2010-07-31T16:25:41+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=597ccddb39c625812157db7ab4bda4b4209021ac'/>
<id>597ccddb39c625812157db7ab4bda4b4209021ac</id>
<content type='text'>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27641 a1c6a512-1295-4272-9138-f99709370657
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27641 a1c6a512-1295-4272-9138-f99709370657
</pre>
</div>
</content>
</entry>
<entry>
<title>FS#11470 - new skin code, finally svn uses the new parser from the theme editor. This means that a skin that passes the editor WILL pass svn and checkwps (unless the target runs out of skin buffer or something.</title>
<updated>2010-07-29T12:37:48+00:00</updated>
<author>
<name>Jonathan Gordon</name>
<email>rockbox@jdgordon.info</email>
</author>
<published>2010-07-29T12:37:48+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=2d31d77a8ba231cb03ec35863c4c4ce2024f6509'/>
<id>2d31d77a8ba231cb03ec35863c4c4ce2024f6509</id>
<content type='text'>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27613 a1c6a512-1295-4272-9138-f99709370657
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27613 a1c6a512-1295-4272-9138-f99709370657
</pre>
</div>
</content>
</entry>
<entry>
<title>Add missing argument that broke building libskin_parser outside of Rockbox.</title>
<updated>2010-07-18T08:58:04+00:00</updated>
<author>
<name>Dominik Riebeling</name>
<email>Dominik.Riebeling@gmail.com</email>
</author>
<published>2010-07-18T08:58:04+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=3cd1968cbc30d32b6d883792c841c897c872860a'/>
<id>3cd1968cbc30d32b6d883792c841c897c872860a</id>
<content type='text'>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27478 a1c6a512-1295-4272-9138-f99709370657
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@27478 a1c6a512-1295-4272-9138-f99709370657
</pre>
</div>
</content>
</entry>
<entry>
<title>Move the skin parser to a seperate library</title>
<updated>2010-06-17T06:52:02+00:00</updated>
<author>
<name>Jonathan Gordon</name>
<email>rockbox@jdgordon.info</email>
</author>
<published>2010-06-17T06:52:02+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=36b934d241d2560be6693f90c9aba501a1ec0ae7'/>
<id>36b934d241d2560be6693f90c9aba501a1ec0ae7</id>
<content type='text'>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@26877 a1c6a512-1295-4272-9138-f99709370657
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@26877 a1c6a512-1295-4272-9138-f99709370657
</pre>
</div>
</content>
</entry>
</feed>
