From 25653e6178ea5261459b8e51d03dbf45fd9cbd43 Mon Sep 17 00:00:00 2001
From: AFCMS <afcms@noreply.git.minetest.land>
Date: Thu, 18 Feb 2021 15:31:13 +0000
Subject: [PATCH] replace pairs() by ipairs() in mcl_playerinfo

---
 mods/PLAYER/mcl_playerinfo/init.lua | 182 ++++++++++++++--------------
 1 file changed, 91 insertions(+), 91 deletions(-)

diff --git a/mods/PLAYER/mcl_playerinfo/init.lua b/mods/PLAYER/mcl_playerinfo/init.lua
index 31dad39a6..4c1454d95 100644
--- a/mods/PLAYER/mcl_playerinfo/init.lua
+++ b/mods/PLAYER/mcl_playerinfo/init.lua
@@ -1,91 +1,91 @@
--- Player state for public API
-mcl_playerinfo = {}
-
--- Get node but use fallback for nil or unknown
-local function node_ok(pos, fallback)
-
-	fallback = fallback or "air"
-
-	local node = minetest.get_node_or_nil(pos)
-
-	if not node then
-		return fallback
-	end
-
-	if minetest.registered_nodes[node.name] then
-		return node.name
-	end
-
-	return fallback
-end
-
-local time = 0
-
-local get_player_nodes = function(player_pos)
-	local work_pos = table.copy(player_pos)
-
-	-- what is around me?
-	work_pos.y = work_pos.y - 0.1 -- standing on
-	local node_stand = node_ok(work_pos)
-	local node_stand_below = node_ok({x=work_pos.x, y=work_pos.y-1, z=work_pos.z})
-
-	work_pos.y = work_pos.y + 1.5 -- head level
-	local node_head = node_ok(work_pos)
-
-	work_pos.y = work_pos.y - 1.2 -- feet level
-	local node_feet = node_ok(work_pos)
-
-	return node_stand, node_stand_below, node_head, node_feet
-end
-
-minetest.register_globalstep(function(dtime)
-
-	time = time + dtime
-
-	-- Run the rest of the code every 0.5 seconds
-	if time < 0.5 then
-		return
-	end
-
-	-- reset time for next check
-	-- FIXME: Make sure a regular check interval applies
-	time = 0
-
-	-- check players
-	for _,player in pairs(minetest.get_connected_players()) do
-		-- who am I?
-		local name = player:get_player_name()
-
-		-- where am I?
-		local pos = player:get_pos()
-
-		-- what is around me?
-		local node_stand, node_stand_below, node_head, node_feet = get_player_nodes(pos)
-		mcl_playerinfo[name].node_stand = node_stand
-		mcl_playerinfo[name].node_stand_below = node_stand_below
-		mcl_playerinfo[name].node_head = node_head
-		mcl_playerinfo[name].node_feet = node_feet
-
-	end
-
-end)
-
--- set to blank on join (for 3rd party mods)
-minetest.register_on_joinplayer(function(player)
-	local name = player:get_player_name()
-
-	mcl_playerinfo[name] = {
-		node_head = "",
-		node_feet = "",
-		node_stand = "",
-		node_stand_below = "",
-	}
-
-end)
-
--- clear when player leaves
-minetest.register_on_leaveplayer(function(player)
-	local name = player:get_player_name()
-
-	mcl_playerinfo[name] = nil
-end)
+-- Player state for public API
+mcl_playerinfo = {}
+
+-- Get node but use fallback for nil or unknown
+local function node_ok(pos, fallback)
+
+	fallback = fallback or "air"
+
+	local node = minetest.get_node_or_nil(pos)
+
+	if not node then
+		return fallback
+	end
+
+	if minetest.registered_nodes[node.name] then
+		return node.name
+	end
+
+	return fallback
+end
+
+local time = 0
+
+local get_player_nodes = function(player_pos)
+	local work_pos = table.copy(player_pos)
+
+	-- what is around me?
+	work_pos.y = work_pos.y - 0.1 -- standing on
+	local node_stand = node_ok(work_pos)
+	local node_stand_below = node_ok({x=work_pos.x, y=work_pos.y-1, z=work_pos.z})
+
+	work_pos.y = work_pos.y + 1.5 -- head level
+	local node_head = node_ok(work_pos)
+
+	work_pos.y = work_pos.y - 1.2 -- feet level
+	local node_feet = node_ok(work_pos)
+
+	return node_stand, node_stand_below, node_head, node_feet
+end
+
+minetest.register_globalstep(function(dtime)
+
+	time = time + dtime
+
+	-- Run the rest of the code every 0.5 seconds
+	if time < 0.5 then
+		return
+	end
+
+	-- reset time for next check
+	-- FIXME: Make sure a regular check interval applies
+	time = 0
+
+	-- check players
+	for _,player in ipairs(minetest.get_connected_players()) do
+		-- who am I?
+		local name = player:get_player_name()
+
+		-- where am I?
+		local pos = player:get_pos()
+
+		-- what is around me?
+		local node_stand, node_stand_below, node_head, node_feet = get_player_nodes(pos)
+		mcl_playerinfo[name].node_stand = node_stand
+		mcl_playerinfo[name].node_stand_below = node_stand_below
+		mcl_playerinfo[name].node_head = node_head
+		mcl_playerinfo[name].node_feet = node_feet
+
+	end
+
+end)
+
+-- set to blank on join (for 3rd party mods)
+minetest.register_on_joinplayer(function(player)
+	local name = player:get_player_name()
+
+	mcl_playerinfo[name] = {
+		node_head = "",
+		node_feet = "",
+		node_stand = "",
+		node_stand_below = "",
+	}
+
+end)
+
+-- clear when player leaves
+minetest.register_on_leaveplayer(function(player)
+	local name = player:get_player_name()
+
+	mcl_playerinfo[name] = nil
+end)