Mapgen: Remove unused 'flat' and 'trees' flags from mg_flags
authorparamat <mat.gregory@virginmedia.com>
Fri, 29 Jul 2016 02:44:58 +0000 (03:44 +0100)
committerparamat <mat.gregory@virginmedia.com>
Thu, 4 Aug 2016 05:58:42 +0000 (06:58 +0100)
commitea12da939fdb0a8fd13de885d104af3031ffc3ac
tree5ceba6e5172e51859d2c0191f69784d2a5631af3
parentfca8e53842718f7f23e11587ae6235c072892b97
Mapgen: Remove unused 'flat' and 'trees' flags from mg_flags

When the 'flat' and 'trees' flags were moved into mgv6_spflags they
were left in mg_flags in an attempt to support old mgv6 worlds. However
their appearence in mg_flags causes confusion, also, later, old-world
support was found to be broken for mgv6 worlds with 'notrees'.

This commit cleans up the mess and comes a month after a thread warning
of the change, and explaining the required action, was posted in the
news subforum. Only old mgv6 worlds with 'flat' or 'notrees' are
affected, a small minority of worlds, the required action being
correctly setting these flags in mgv6_spflags.

Disable a section of the 'map settings manager' unit test which is to
be changed as it is causing problems for pull requests.
src/mapgen.cpp
src/mapgen.h
src/mapgen_v6.cpp
src/unittest/test_map_settings_manager.cpp