Code review comment for lp:~widelands-dev/widelands/new_starting_conditions

Revision history for this message
GunChleoc (gunchleoc) wrote :

> > But they do get positioned right next to the headquarters, so the gain is
> very little...
>
> I mostly agree, some function to place them in bigger distance would be useful
> here

Since we're placing them last now, they get placed at the edge of the terrain, so there is some gain :)

> > When you next have some time, it would be great if you could look at some of
> the merge requests that kaputtnik has already approved
>
> I wondered why there are approved branches sitting that long - I will have a
> look, of course....

Thanks!

« Back to merge proposal