From 431d8a9b83be858193328fe59e75026fa023393f Mon Sep 17 00:00:00 2001 From: HybridDog Date: Thu, 7 Mar 2019 08:31:25 +0100 Subject: Abort when trying to set a not registered node (#7011) I removed the MapNode constructor which takes a nodename and gives the node's id or CONTENT_IGNORE The code which used this constructor (two places) now handles the situation of not registered nodes correctly: * minetest.set_node and similar functions make minetest crash when a not registered node is passed * reverting a node with rollback aborts if the node is not registered --- src/mapnode.h | 5 ----- 1 file changed, 5 deletions(-) (limited to 'src/mapnode.h') diff --git a/src/mapnode.h b/src/mapnode.h index 6e48bce8c..7a3d30ddc 100644 --- a/src/mapnode.h +++ b/src/mapnode.h @@ -145,11 +145,6 @@ struct MapNode param2(a_param2) { } - // Create directly from a nodename - // If name is unknown, sets CONTENT_IGNORE - MapNode(const NodeDefManager *ndef, const std::string &name, - u8 a_param1=0, u8 a_param2=0); - bool operator==(const MapNode &other) const noexcept { return (param0 == other.param0 -- cgit v1.2.3