- Status Closed
- Percent Complete
- Task Type Patches
- Category User Interface → Themes
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Version 3.3
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10462 - WPS plumbing
This patch is an attempt to get the wps code more split up into the wps engine and the wps screen… What I’d like to see is for the WPS screen code to only have a single call into the wps engine code which is to do the drawing… likewise, the engine code should have absolutly nothing to do with playback and changing any state of anything..
This patch doesn’t add or remove any code (except a couple of headers), it only moves code around a bit… feel free to suggest a better naming scheme
The next thing to do would be split gwps.h up into whats actually needed by external code (pretty much nothing but a wps_redraw() header) and whats needed by the engine code…
after that, gwps.c and gwps-common.c could be merged into one as they both now only deal with changing playback behaviour (there are a few functions which I wasnt sure right now where the best place for them was so there is a bit which could still be moved.)
2009-07-27 07:35
Reason for closing: Out of Date
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
done in r22062
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
I think you forgot svn add’ing the new files?
grr… I did svn cp.. doesnt that add them automatically?