Stop setting landparam in Hunting Ground generation

zenorogue / hyperrogue

A SDL roguelike in a non-euclidean world

GNU General Public License v2.0

569 stars 72 forks source link

Closed jruderman closed 3 years ago

jruderman commented 3 years ago

Seems to be unused: I didn't see any code related to Hunting Ground or dogs that checks landparam.
Could leak into other lands: for example, with ./hyper -C -W Hunting -W2 Minefield -I Turquoise 1, some dogs that generate on Minefield cells would also incorrectly mark their Minefield cell as "known mine".

zenorogue commented 3 years ago

Seems unused indeed (maybe some old debug?) but I added checks if(dog2->land == laHunting) for safety.