Rockbox

safetydan - are you still working on this?

I’m currently in the progress in looking deeper and deeper into the current wps code.
And the deeper I look into, the more do I think that the WPS parsing code needs complete reworking

I had a look at your code and it looks much cleaner.
It seems to be a much better approach than the current code!

I have some additional ideas which we could discuss.
When are you online in IRC?
Or would a forum’s talk be better for you?

I haven’t looked at it since I posted this task, but I agree it’s a cleaner approach :) I’ll be on IRC once I’m fully back from holidays.

Here is my first version of a token based parser.
I used your code as base and changed it a lot

I used less token types and tried to put all available tokens in an array
which also contain fucntion pointers for special parsing (if needed - there’s
also a default parsing function which should work for 90% of all tags) and
there are action functions which later will be called by the wps execution
code to do the “work” for the tag (e.g. load an image and display it).

My code is able to parse basic WPS including conditionals and also conditionals
inside conditionals - but only if the conditional arguments consists only
of one token!
e.g. “%?ps<Shuffle Off|%?mh<Shuffle On|Shuffle and Hold On»” will work

   "%?ps<Shuffle Off|Shuffle%?mh<| and Hold> On>>" will not work

(second argument off %?ps contains two strings and a sub-conditional)

I didn’t had a look at sublines - so I assume they will not work ;)

There are still a _lot_ TODO’s, but my plan is to finish it and create
a working patch for the CVS - but this may take some time…

safetydan, I would be glad if you (or anybody else who wants to) could
have a look at my code and maybe have some ideas for a better handling
of the conditionals/subconditionals and their arguments.

I’m sure you’ll find a lot of other things to improve (at least I do ;)
- it’ll not be the latest incarnation of a token based parsing code

   gwps-tokenizer_MM-20060828.c (39.5 KiB)

No one wants to test and comment it?

That’s really sad

I’m currently in the process of reworking this, using bits of both Dan’s and Matthias’s code. I think I’ll have something presentable quite soon.

So here’s my code. I used the ideas I found most interesting in both the files above to try to make everything as simple as possible.
Basically it works like Dan’s but should print a correct parse tree, and it uses Matthias’s idea of an array containing all the different possible tags. This makes the code much shorter and much easier to maintain.
Also I added support for alternating sublines.
When I have time I’ll try to make this into a usable patch.

Please tell me what you think about it :)

   gwps-tokenizer_Nico.c (17.3 KiB)