Fix #184, Fix #186, Fix #187

Just some minor issues like dead code.
minetest-mods · Nov 21, 2014 · dcf1f79 · dcf1f79
1 parent d325292
commit dcf1f79
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/mesecons/actionqueue.lua b/mesecons/actionqueue.lua
@@ -44,7 +44,8 @@ end
 -- However, even that does not work in some cases, that's why we delay the time the globalsteps
 -- start to be execute by 5 seconds
 local get_highest_priority = function (actions)
-	local highestp = -1, highesti
+	local highestp = -1
+	local highesti
 	for i, ac in ipairs(actions) do
 		if ac.priority > highestp then
 			highestp = ac.priority

diff --git a/mesecons_luacontroller/init.lua b/mesecons_luacontroller/init.lua
@@ -477,7 +477,6 @@ minetest.register_node(nodename, {
 			reset_meta(pos, fields.code, err)
 		end
 	end,
-	on_timer = handle_timer,
 	sounds = default.node_sound_stone_defaults(),
 	mesecons = mesecons,
 	digiline = digiline,

diff --git a/mesecons_pistons/init.lua b/mesecons_pistons/init.lua
@@ -56,14 +56,15 @@ end
 
 local piston_remove_pusher = function(pos, node)
 	pistonspec = minetest.registered_nodes[node.name].mesecons_piston
-	if pushername == pistonspec.pusher then --make sure there actually is a pusher (for compatibility reasons mainly)
-		return
-	end
-
 	dir = piston_get_direction(pistonspec.dir, node)
 	local pusherpos = mesecon:addPosRule(pos, dir)
 	local pushername = minetest.get_node(pusherpos).name
 
+	-- make sure there actually is a pusher (for compatibility reasons mainly)
+	if pushername ~= pistonspec.pusher then
+		return
+	end
+
 	minetest.remove_node(pusherpos)
 	minetest.sound_play("piston_retract", {
 		pos = pos,