Code review comment for lp:~widelands-dev/widelands/bug_1571009_work_area_radius
Revision history for this message
| GunChleoc (gunchleoc) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
I still don't agree with your TODO comment. The Fortress has multiple workareas on purpose - it shows you the area that it will be able to conquer when you upgrade to a Citadel. If you have ideas for improving the code, replacing the map with a uint is not the way to go.