AliYousuf / hedgewars

Automatically exported from code.google.com/p/hedgewars
GNU General Public License v2.0
0 stars 0 forks source link

DiagonalMaze_v1 is broken #844

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Obtain Hedgewars git b13ad84…
2. Somehow obtain the script “DiagonalMaze_v1”
3. Look on the map preview
4. Start the game

What is the expected output? What do you see instead?
Expected: The map preview looks like a maze and if you start the game it is a 
maze.
Actual outcome: The map preview looks like the perlin noise random map 
generator has been used. If you start the game, the preview wasn’t lying to 
you.

What version of the product are you using? On what operating system?
git b13ad84… on GNU/linux

Please provide any additional information below.
DiagonalMaze_v1 used to work around Easter 2014, this was even promoted to the 
front page.

Original issue reported on code.google.com by almikes@aol.com on 2 Dec 2014 at 5:55

GoogleCodeExporter commented 8 years ago
Thanks for making an issue for this.

This was already discussed on IRC, here's just as related, general note I want 
to leave:

Please use the new globals mgRandom, mgMaze, mgPerlin or mgDrawn as mapgen, 
instead of semi-"arbitrary" numbers, in future Lua Scripts.

This will ensure future script compatibility even if somebody should happen to 
change the internal implementation of mapgens again.

Thank you.

Btw: Is there any reason why diagonal mazes are not included in the repo yet? :)

Original comment by sheepyluva on 2 Dec 2014 at 6:06

GoogleCodeExporter commented 8 years ago
fixed in rf2f9928ea2e7346a905fc78ae9e1a901670568ad
also: DiagonalMaze was added to repo

Original comment by sheepyluva on 2 Dec 2014 at 8:59