Code review comment for lp:~widelands-dev/widelands/bug-1783878_editor_random_map_tribe

GunChleoc (gunchleoc) wrote :

@ypopezios: Good idea, but not as easy as it sounds - when set to "random", the tribe is already sneakily assigned in the background so that the user can choose an initialization ("Headquarters") etc. So, removing that extra variable has a lot of other implications in the code as well and is not trivial at all. So, definitely not for Build 20 ;)

« Back to merge proposal