Code review comment for lp:~widelands-dev/widelands/cleanup-statistics-plotter
Revision history for this message
| GunChleoc (gunchleoc) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
f33e652
(Get the code!)
Since we're down to 1 bug to fix for Build 20, feature freeze needs to be taken very seriously. Any innocent-looking line of code can potentially introduce a breaking bug that would force us to postpone Build 20.
Code LGTM :)
I think having the min value in the statistics from newstats_1 is a good idea, because the bottom value is not 0. Since it bothers hessenfarmer, maybe there is a way to display both?