Opened 12 years ago
Last modified 12 years ago
#1688 closed enhancement
[PATCH] Add random map "Belgian Uplands" based on erosion — at Version 1
| Reported by: | FeXoR | Owned by: | |
|---|---|---|---|
| Priority: | Should Have | Milestone: | Alpha 12 |
| Component: | UI & Simulation | Keywords: | patch RMS RMGEN |
| Cc: | Patch: |
Description (last modified by )
A random map with its heightmap generated by erosion. It will not be as fair as more symmetric maps but looks more natural.
Change History (1)
comment:1 by , 12 years ago
| Description: | modified (diff) |
|---|---|
| Keywords: | patch added; random map erosion hightmap removed |
Note:
See TracTickets
for help on using tickets.
From a quick look at the patch, I see at least two unnecessary
deepcopy()s, on line 314 snd 324 ofbelgian_uplands.js(since the assigned variable is just a number and not an object reference).