From 99a54319b108ffd91fc233bf8c444ad6a7f3da7a Mon Sep 17 00:00:00 2001
From: Wuzzy <almikes@aol.com>
Date: Wed, 24 May 2017 12:15:20 +0200
Subject: [PATCH] Refactor mcl_dungeons

---
 mods/MAPGEN/mcl_dungeons/init.lua | 55 ++++++++++++++++---------------
 1 file changed, 29 insertions(+), 26 deletions(-)

diff --git a/mods/MAPGEN/mcl_dungeons/init.lua b/mods/MAPGEN/mcl_dungeons/init.lua
index fa7bba8b12..726383c303 100644
--- a/mods/MAPGEN/mcl_dungeons/init.lua
+++ b/mods/MAPGEN/mcl_dungeons/init.lua
@@ -55,7 +55,7 @@ local get_loot = function()
 
 	return items
 end
-	
+
 
 -- Buffer for LuaVoxelManip
 local lvm_buffer = {}
@@ -153,31 +153,34 @@ minetest.register_on_generated(function(minp, maxp)
 		end
 
 		-- Check conditions. If okay, start generating
-
-		-- But prepare random chest positions beforehand
-		-- We assign each position at the wall a number and each chest gets one of these numbers randomly
-		local totalChests = 2
-		local totalChestSlots = (dim.x-1) * (dim.z-1)
-		local chestSlots = {}
-		-- There is a small chance that both chests have the same slot.
-		-- In that case, we give a 2nd chance for the 2nd chest to get spawned.
-		-- If it failed again, tough luck! We stick with only 1 chest spawned.
-		local lastRandom
-		local secondChance = true -- second chance is still available
-		for i=1, totalChests do
-			local r = math.random(1, totalChestSlots)
-			if r == lastRandom and secondChance then
-				-- Oops! Same slot selected. Try again.
-				r = math.random(1, totalChestSlots)
-				secondChance = false
-			end
-			lastRandom = r
-			table.insert(chestSlots, r)
-		end
-		table.sort(chestSlots)
-		local currentChest = 1
-
 		if ceilingfloor_ok and openings >= 1 and openings <= 5 then
+			-- Okay! Spawning starts!
+
+			-- First prepare random chest positions.
+			-- Chests spawn at wall
+
+			-- We assign each position at the wall a number and each chest gets one of these numbers randomly
+			local totalChests = 2
+			local totalChestSlots = (dim.x-1) * (dim.z-1)
+			local chestSlots = {}
+			-- There is a small chance that both chests have the same slot.
+			-- In that case, we give a 2nd chance for the 2nd chest to get spawned.
+			-- If it failed again, tough luck! We stick with only 1 chest spawned.
+			local lastRandom
+			local secondChance = true -- second chance is still available
+			for i=1, totalChests do
+				local r = math.random(1, totalChestSlots)
+				if r == lastRandom and secondChance then
+					-- Oops! Same slot selected. Try again.
+					r = math.random(1, totalChestSlots)
+					secondChance = false
+				end
+				lastRandom = r
+				table.insert(chestSlots, r)
+			end
+			table.sort(chestSlots)
+			local currentChest = 1
+
 			-- Ceiling and floor
 			local maxx, maxy, maxz = x+dim.x+1, y+dim.y+1, z+dim.z+1
 			local chestSlotCounter = 1
@@ -185,7 +188,7 @@ minetest.register_on_generated(function(minp, maxp)
 				for tz = z, maxz do
 					for ty = y, maxy do
 						local p_pos = area:index(tx, ty, tz)
-	
+
 						-- Floor
 						if ty == y then
 							if math.random(1,4) == 1 then