Review: Resubmit Hello Gun, I reviewd that comment in workarea_info.h again. Its just that I do not fully understand the usage of that map, yet. So if you know what stings are used, please add them.
As of the implementation of std:map this will be correct, I guess. But it usually is a abd idea to depend on some implemenation that may change. OTOH the hashcode of integer may not change ever again. Example: From Java JDK1.3 to 1.4 the Implementation of String.hashCode() was changed. Some code had implicit dependencies on the order of Elements in string based maps an therefore started to fail. Please feel free to change the comment to your liking. -- https://code.launchpad.net/~widelands-dev/widelands/bug_1571009_work_area_radius/+merge/292066 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug_1571009_work_area_radius. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
