Closed ghost closed 12 years ago
Thanks for the patch.
How well has it been tested? I never took the time to understand this code originally. If it's been tested at least once or twice I'll just merge immediately.
I plan to review the share-building progress monitor more thoroughly though.
Yeah I have given it a couple of run throughs. I can't say I've done detailed testing on it, but some light acceptance testing would suggest it is fine.
Good enough for a graph :)
Deleted merged branch.
Issue 19: The speed graph used to end early on the left hand side. Fixed the logic so now it correctly fills across to the left. Also fixed a number of stupid code constructs so its much easier to read and understand now, as well as being more efficient.