From 936d67dad4c4939c9f7c30f489d8bfc74f47fcaf Mon Sep 17 00:00:00 2001 From: paramat Date: Thu, 22 Jun 2017 06:50:22 +0100 Subject: Ores: Make 'absheight' flag non-functional The 'absheight' flag was added years ago for the floatlands of 'indev' mapgen (now deleted). The feature mirrored all ore placement around y = 0 to place ores in floatlands. In MTG we now use dedicated ore registrations for floatlands. The feature is crude, inflexible, problematic and very rarely used, it also makes ore vertical range code more complex. Minetest 0.5 is a good chance to remove the feature. The flag itself remains to not break flag values. --- src/mg_ore.cpp | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) (limited to 'src/mg_ore.cpp') diff --git a/src/mg_ore.cpp b/src/mg_ore.cpp index f959ca9e6..73af2e2e6 100644 --- a/src/mg_ore.cpp +++ b/src/mg_ore.cpp @@ -27,7 +27,7 @@ with this program; if not, write to the Free Software Foundation, Inc., FlagDesc flagdesc_ore[] = { - {"absheight", OREFLAG_ABSHEIGHT}, + {"absheight", OREFLAG_ABSHEIGHT}, // Non-functional {"puff_cliffs", OREFLAG_PUFF_CLIFFS}, {"puff_additive_composition", OREFLAG_PUFF_ADDITIVE}, {NULL, 0} @@ -87,22 +87,11 @@ void Ore::resolveNodeNames() size_t Ore::placeOre(Mapgen *mg, u32 blockseed, v3s16 nmin, v3s16 nmax) { - int in_range = 0; - - in_range |= (nmin.Y <= y_max && nmax.Y >= y_min); - if (flags & OREFLAG_ABSHEIGHT) - in_range |= (nmin.Y >= -y_max && nmax.Y <= -y_min) << 1; - if (!in_range) + if (!(nmin.Y <= y_max && nmax.Y >= y_min)) return 0; - int actual_ymin, actual_ymax; - if (in_range & ORE_RANGE_MIRROR) { - actual_ymin = MYMAX(nmin.Y, -y_max); - actual_ymax = MYMIN(nmax.Y, -y_min); - } else { - actual_ymin = MYMAX(nmin.Y, y_min); - actual_ymax = MYMIN(nmax.Y, y_max); - } + int actual_ymin = MYMAX(nmin.Y, y_min); + int actual_ymax = MYMIN(nmax.Y, y_max); if (clust_size >= actual_ymax - actual_ymin + 1) return 0; -- cgit v1.2.3