Beanzilla/chunkkeeper-cd2025
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed possible strang occurance of segfault

Browse Source 
I just encountered a segfault while using this mod.

  It could be that this mod had something to do with it, or perhaps not.

  So with the thought of reducing heavy strain I've come up with not
changing the persistance of the node unless we really have to (i.e.
swap_node then change persistance), this should reduce possible bugs
within the Minetest server thus more reliable/better. (Either way this
code change should help greatly in stability)
This commit is contained in:
Apollo 2022-08-28 23:44:45 -04:00
parent e6db8d8e04
commit 1868cf815b
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
register.lua
View File

@ -24,6 +24,7 @@ function chunkkeeper.processFuel(pos)
--chunkkeeper.log({timer=timer, super=super, running=run})
dirty = true
minetest.swap_node(pos, {name = "chunkkeeper:keeper_off"})
minetest.forceload_free_block(pos)
end
end
meta:set_int("time_left", timer)
@ -31,13 +32,6 @@ function chunkkeeper.processFuel(pos)
elseif super then -- Just update with super
dirty = true
end
if run then -- Change persistance based on running or not (this could also mean the keeper's running was manually toggled)
minetest.forceload_block(pos)
--chunkkeeper.log({msg="Persisting position", pos=pos})
else
minetest.forceload_free_block(pos)
--chunkkeeper.log({msg="Stopped persisting position", pos=pos})
end
if dirty then
if not super then
@ -301,6 +295,7 @@ minetest.register_node("chunkkeeper:keeper_off", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_off"})
end
minetest.forceload_free_block(pos)
else
meta:set_int("running", 1)
if not super then
@ -308,6 +303,7 @@ minetest.register_node("chunkkeeper:keeper_off", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_on"})
end
minetest.forceload_block(pos)
end
dirty = true
elseif fields.toggle_hide_owner then
@ -423,6 +419,7 @@ minetest.register_node("chunkkeeper:keeper_on", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_off"})
end
minetest.forceload_free_block(pos)
else
meta:set_int("running", 1)
if not super then
@ -430,6 +427,7 @@ minetest.register_node("chunkkeeper:keeper_on", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_on"})
end
minetest.forceload_block(pos)
end
dirty = true
elseif fields.toggle_hide_owner then
@ -507,6 +505,7 @@ minetest.register_node("chunkkeeper:keeper_inf_off", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_off"})
end
minetest.forceload_free_block(pos)
else
meta:set_int("running", 1)
if not super then
@ -514,6 +513,7 @@ minetest.register_node("chunkkeeper:keeper_inf_off", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_on"})
end
minetest.forceload_block(pos)
end
dirty = true
elseif fields.toggle_hide_owner then
@ -598,6 +598,7 @@ minetest.register_node("chunkkeeper:keeper_inf_on", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_off"})
end
minetest.forceload_free_block(pos)
else
meta:set_int("running", 1)
if not super then
@ -605,6 +606,7 @@ minetest.register_node("chunkkeeper:keeper_inf_on", {
else
minetest.swap_node(pos, {name = "chunkkeeper:keeper_inf_on"})
end
minetest.forceload_block(pos)
end
dirty = true
elseif fields.toggle_hide_owner then