Accomplishments' Daemon Code Improvements
There development of the daemon code, up until now, has been mostly monolithic. This needs to be properly Pythonized and Twistified.
Here are some initial thoughts on this:
* currently the daemon.app module is 871 lines long
* many of those lines are longer than 80 characters
* there is dbus-specific stuff in there that should be placed in a
separate module
* method names are mixed lowerCamelCase and under_scored -- we need
to choose one and stick to it, using the same one across the entire
project
* it makes extensive use of inlineCallbacks; these are inherently
slow, and can cause Twisted apps to run as much as 10 times slower
than by simply using deferreds and callbacks; refactor!
* the Accomplishments class is HUGE; some of that code can be split
up into utility functions, smaller methods which isolate logical
chunks, image manipulation code can go elsewhere, config file
manipulation as well,
Some or all of these will be converted to bugs...
Blueprint information
- Status:
- Not started
- Approver:
- None
- Priority:
- Undefined
- Drafter:
- None
- Direction:
- Needs approval
- Assignee:
- Duncan McGreggor
- Definition:
- New
- Series goal:
- None
- Implementation:
- Unknown
- Milestone target:
- None
- Started by
- Completed by