Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: WPS tokenizer

WPS tokenizer

From: Nicolas Pennequin <nicolas.pennequin_at_free.fr>
Date: Sun, 18 Mar 2007 21:43:25 +0100

So I'm finally ready to present a patch for the WPS tokenizer.
The basic idea is to store the WPS as an array of tokens. This idea was
already the base for two tracker entries :
http://www.rockbox.org/tracker/task/2898
http://www.rockbox.org/tracker/task/4826
I used the code in FS#4826 as a base and improved the parser. After that, I
started working on integrating it into the current code. This proved to be
quite hard, so I decided to rewrite some parts of it, namely
gui_wps_refresh() and most of the functions it uses.

You can find the patch here :
http://nicolas.pennequin.free.fr/rockbox/wps_tokenizer.patch
It's a big baby: 155k !

Please tell me what you think of the concept and the code.
IMHO this has the advantage of being much clearer, readable and maintainable
than the current code. As of performance, I don't have the confirmation that
it's faster, but I think it should be, as it was designed to make the parser
do all the work. Binsize isa bit of a disappointment. I thought it could be
smaller, but instead I'm seeing a (small) increase on some targets (I haven't
done much checking, though).

Nicolas
Received on 2007-03-18


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa