[mesecons_delayer] fix code style issues

mods/ITEMS/REDSTONE/mesecons_delayer/init.lua

@@ -4,7 +4,7 @@ local DELAYS = { 0.1, 0.2, 0.3, 0.4 }
 local DEFAULT_DELAY = DELAYS[1]
 
 -- Function that get the input/output rules of the delayer
-local delayer_get_output_rules = function(node)
+local function delayer_get_output_rules(node)
 	local rules = {{x = -1, y = 0, z = 0, spread=true}}
 	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
@@ -12,7 +12,7 @@ local delayer_get_output_rules = function(node)
 	return rules
 end
 
-local delayer_get_input_rules = function(node)
+local function delayer_get_input_rules(node)
 	local rules = {{x = 1, y = 0, z = 0}}
 	for i = 0, node.param2 do
 		rules = mesecon.rotate_rules_left(rules)
@@ -22,7 +22,7 @@ end
 
 -- Return the sides of a delayer.
 -- Those are used to toggle the lock state.
-local delayer_get_sides = function(node)
+local function delayer_get_sides(node)
 	local rules = {
 		{x = 0, y = 0, z = -1},
 		{x = 0, y = 0, z =  1},
@@ -35,7 +35,7 @@ end
 
 -- Make the repeater at pos try to lock any repeater it faces.
 -- Returns true if a repeater was locked.
-local check_lock_repeater = function(pos, node)
+local function check_lock_repeater(pos, node)
 	-- Check the repeater at pos and look if it faces
 	-- a repeater placed sideways.
 	-- If yes, lock the second repeater.
@@ -67,7 +67,7 @@ end
 
 -- Make the repeater at pos try to unlock any repeater it faces.
 -- Returns true if a repeater was unlocked.
-local check_unlock_repeater = function(pos, node)
+local function check_unlock_repeater(pos, node)
 	-- Check the repeater at pos and look if it faces
 	-- a repeater placed sideways.
 	-- If yes, also check if the second repeater doesn't receive
@@ -119,21 +119,19 @@ local check_unlock_repeater = function(pos, node)
 end
 
 -- Functions that are called after the delay time
-local delayer_activate = function(pos, node)
+local function delayer_activate(pos, node)
 	local def = minetest.registered_nodes[node.name]
 	local time = def.delayer_time
 	minetest.set_node(pos, {name=def.delayer_onstate, param2=node.param2})
 	mesecon.queue:add_action(pos, "receptor_on", {delayer_get_output_rules(node)}, time, nil)
-
 	check_lock_repeater(pos, node)
 end
 
-local delayer_deactivate = function(pos, node)
+local function delayer_deactivate(pos, node)
 	local def = minetest.registered_nodes[node.name]
 	local time = def.delayer_time
 	minetest.set_node(pos, {name=def.delayer_offstate, param2=node.param2})
 	mesecon.queue:add_action(pos, "receptor_off", {delayer_get_output_rules(node)}, time, nil)
-
 	check_unlock_repeater(pos, node)
 end
 
@@ -145,44 +143,44 @@ end
 -- Register the 2 (states) x 4 (delay times) delayers
 
 for i = 1, 4 do
-local groups
+	local groups
-if i == 1 then
+	if i == 1 then
 		groups = {dig_immediate=3,dig_by_water=1,destroy_by_lava_flow=1,dig_by_piston=1,attached_node=1,redstone_repeater=i}
-else
+	else
 		groups = {dig_immediate=3,dig_by_water=1,destroy_by_lava_flow=1,dig_by_piston=1,attached_node=1,redstone_repeater=i,not_in_creative_inventory=1}
-end
+	end
 
-local delaytime = DELAYS[i]
+	local delaytime = DELAYS[i]
 
-local boxes
+	local boxes
-if i == 1 then
+	if i == 1 then
 		boxes = {
 			{ -8/16, -8/16, -8/16, 8/16, -6/16, 8/16 },		-- the main slab
 			{ -1/16, -6/16, 6/16, 1/16, -1/16, 4/16},     -- still torch
 			{ -1/16, -6/16, 0/16, 1/16, -1/16, 2/16},     -- moved torch
 		}
-elseif i == 2 then
+	elseif i == 2 then
 		boxes = {
 			{ -8/16, -8/16, -8/16, 8/16, -6/16, 8/16 },		-- the main slab
 			{ -1/16, -6/16, 6/16, 1/16, -1/16, 4/16},     -- still torch
 			{ -1/16, -6/16, -2/16, 1/16, -1/16, 0/16},     -- moved torch
 		}
-elseif i == 3 then
+	elseif i == 3 then
 		boxes = {
 			{ -8/16, -8/16, -8/16, 8/16, -6/16, 8/16 },		-- the main slab
 			{ -1/16, -6/16, 6/16, 1/16, -1/16, 4/16},     -- still torch
 			{ -1/16, -6/16, -4/16, 1/16, -1/16, -2/16},     -- moved torch
 		}
-elseif i == 4 then
+	elseif i == 4 then
 		boxes = {
 			{ -8/16, -8/16, -8/16, 8/16, -6/16, 8/16 },		-- the main slab
 			{ -1/16, -6/16, 6/16, 1/16, -1/16, 4/16},     -- still torch
 			{ -1/16, -6/16, -6/16, 1/16, -1/16, -4/16},     -- moved torch
 		}
-end
+	end
 
-local help, tt, longdesc, usagehelp, icon, on_construct
+	local help, tt, longdesc, usagehelp, icon, on_construct
-if i == 1 then
+	if i == 1 then
 		help = true
 		tt = S("Transmits redstone power only in one direction").."\n"..
 			S("Delays signal").."\n"..
@@ -191,7 +189,6 @@ if i == 1 then
 		usagehelp = S("To power a redstone repeater, send a signal in “arrow” direction (the input). The signal goes out on the opposite side (the output) with a delay. To change the delay, use the redstone repeater. The delay is between 0.1 and 0.4 seconds long and can be changed in steps of 0.1 seconds. It is indicated by the position of the moving redstone torch.").."\n"..
 				S("To lock a repeater, send a signal from an adjacent repeater into one of its sides. While locked, the moving redstone torch disappears, the output doesn't change and the input signal is ignored.")
 		icon = "mesecons_delayer_item.png"
-
 		-- Check sides of constructed repeater and lock it, if required
 		on_construct = function(pos)
 			local node = minetest.get_node(pos)
@@ -219,18 +216,18 @@ if i == 1 then
 				end
 			end
 		end
-else
+	else
 		help = false
-end
+	end
 
-local desc_off
+	local desc_off
-if i == 1 then
+	if i == 1 then
 		desc_off = S("Redstone Repeater")
-else
+	else
 		desc_off = S("Redstone Repeater (Delay @1)", i)
-end
+	end
 
-minetest.register_node("mesecons_delayer:delayer_off_"..tostring(i), {
+	minetest.register_node("mesecons_delayer:delayer_off_"..tostring(i), {
 		description = desc_off,
 		inventory_image = icon,
 		wield_image = icon,
@@ -290,22 +287,19 @@ minetest.register_node("mesecons_delayer:delayer_off_"..tostring(i), {
 		delayer_lockstate = "mesecons_delayer:delayer_off_locked",
 		sounds = mcl_sounds.node_sound_stone_defaults(),
 		mesecons = {
-		receptor =
+			receptor = {
-		{
 				state = mesecon.state.off,
-			rules = delayer_get_output_rules
+				rules = delayer_get_output_rules,
 			},
-		effector =
+			effector = {
-		{
 				rules = delayer_get_input_rules,
-			action_on = delayer_activate
+				action_on = delayer_activate,
-		}
+			},
 		},
 		on_rotate = on_rotate,
-})
+	})
 
-
+	minetest.register_node("mesecons_delayer:delayer_on_"..tostring(i), {
-minetest.register_node("mesecons_delayer:delayer_on_"..tostring(i), {
 		description = S("Redstone Repeater (Delay @1, Powered)", i),
 		_doc_items_create_entry = false,
 		drawtype = "nodebox",
@@ -343,6 +337,7 @@ minetest.register_node("mesecons_delayer:delayer_on_"..tostring(i), {
 				minetest.record_protection_violation(pos, protname)
 				return
 			end
+			--HACK! we already know the node name, so we should generate the function to avoid multiple checks
 			if node.name=="mesecons_delayer:delayer_on_1" then
 				minetest.set_node(pos, {name="mesecons_delayer:delayer_on_2",param2=node.param2})
 			elseif node.name=="mesecons_delayer:delayer_on_2" then
@@ -361,20 +356,17 @@ minetest.register_node("mesecons_delayer:delayer_on_"..tostring(i), {
 		delayer_lockstate = "mesecons_delayer:delayer_on_locked",
 		sounds = mcl_sounds.node_sound_stone_defaults(),
 		mesecons = {
-		receptor =
+			receptor = {
-		{
 				state = mesecon.state.on,
-			rules = delayer_get_output_rules
+				rules = delayer_get_output_rules,
 			},
-		effector =
+			effector = {
-		{
 				rules = delayer_get_input_rules,
-			action_off = delayer_deactivate
+				action_off = delayer_deactivate,
-		}
+			},
 		},
 		on_rotate = on_rotate,
-})
+	})
-
 end