draw_debug_polygons() now handles parenting the line mesh all on its own.

- Character3D prevent crashes on _ready by checking for a valid skeleton.
- Fixed deadzone reading as cleared for all axes when only one has done so.
- Singletons once again deregistered, since it seems to now cause less crashes.
- PlayerSpawn now uses the new error-printing macro.

src/liborng/nodes/character_nodes/character_3d.cpp

@@ -20,15 +20,24 @@ using namespace godot;
 
 void Character3D::_ready()
 {
-	TypedArray<Node> child_mesh_candidates = this->skeleton->get_children();
+	if (this->skeleton)
-	const uint32_t num_children = child_mesh_candidates.size();
-	for (uint32_t i = 0; i < num_children; i++)
 	{
-		if (MeshInstance3D *mesh = cast_to<MeshInstance3D>(child_mesh_candidates[i]))
+		TypedArray<Node> child_mesh_candidates = this->skeleton->get_children();
+		const uint32_t num_children = child_mesh_candidates.size();
+		for (uint32_t i = 0; i < num_children; i++)
 		{
-			this->meshes.emplace_back(mesh);
+			if (MeshInstance3D *mesh = cast_to<MeshInstance3D>(child_mesh_candidates[i]))
+			{
+				this->meshes.emplace_back(mesh);
+			}
 		}
 	}
+#ifdef DEBUG_ENABLED
+	else
+	{
+		PRINT_ERROR(CHARACTER3D_TAG, "No skeleton has been set, and there is currently no way to seek one out in the hierarchy. This will be a problem.");
+	}
+#endif // DEBUG_ENABLED
 }
 
 
@@ -55,16 +64,19 @@ void Character3D::take_damage(const Dictionary &hit_data)
 
 void Character3D::_activate_ragdoll()
 {
-	const TypedArray<Node> &physical_bones = this->physical_bone_simulator->get_children();
+	if (this->physical_bone_simulator)
-	const uint32_t num_physical_bones = physical_bones.size();
-	for (uint32_t i = 0; i < num_physical_bones; i++)
 	{
-		if (PhysicalBone3D *bone = cast_to<PhysicalBone3D>(physical_bones[i]))
+		const TypedArray<Node> &physical_bones = this->physical_bone_simulator->get_children();
+		const uint32_t num_physical_bones = physical_bones.size();
+		for (uint32_t i = 0; i < num_physical_bones; i++)
 		{
-			bone->set_collision_layer(0);
+			if (PhysicalBone3D *bone = cast_to<PhysicalBone3D>(physical_bones[i]))
+			{
+				bone->set_collision_layer(0);
+			}
 		}
+		this->physical_bone_simulator->physical_bones_start_simulation();
 	}
-	this->physical_bone_simulator->physical_bones_start_simulation();
 }
 
 

src/liborng/nodes/hurtbox/hurtbox_base.cpp

@@ -10,6 +10,7 @@
 
 #include "character_3d.h"
 
+#include <godot_cpp/classes/engine.hpp>
 #include <godot_cpp/classes/physical_bone3d.hpp>
 #include <godot_cpp/classes/scene_tree.hpp>
 #include <godot_cpp/classes/scene_tree_timer.hpp>
@@ -17,8 +18,35 @@
 using namespace godot;
 
 
+void HurtboxBase::_enter_tree()
+{
+	if (!this->collision_shape)
+	{
+		this->collision_shape = memnew(CollisionShape3D);
+		this->collision_shape->set_shape(this->shape);
+		this->add_child(this->collision_shape);
+	}
+	
+	Area3D::_enter_tree();
+}
+
+void HurtboxBase::_exit_tree()
+{
+	if (this->collision_shape)
+	{
+		this->collision_shape->set_shape(this->shape);
+		this->remove_child(this->collision_shape);
+		this->collision_shape->queue_free();
+	}
+	
+	Area3D::_exit_tree();
+}
+
 void HurtboxBase::_ready()
 {
+#ifdef DEBUG_ENABLED
+	if(Engine::get_singleton()->is_editor_hint()) { this->set_physics_process(false); return; }
+#endif	// DEBUG_ENABLED
 	this->set_process(false);
 	this->remaining_life = this->lifetime;
 	
@@ -27,11 +55,6 @@ void HurtboxBase::_ready()
 
 void HurtboxBase::_physics_process(double delta)
 {
-	if (this->remaining_life <= 0.0f)
-	{
-		this->collision_shape->set_disabled(true);
-	}
-	
 	TypedArray<Character3D> characters_hit_this_frame;
 	
 	const TypedArray<Node3D> &bodies = this->get_overlapping_bodies();
@@ -73,7 +96,10 @@ void HurtboxBase::_physics_process(double delta)
 	
 	this->hit_character_bodies.append_array(characters_hit_this_frame);
 	
-	this->remaining_life -= delta;
+	if (this->remaining_life <= 0.0f)
+		this->collision_shape->set_disabled(true);
+	else
+		this->remaining_life -= delta;
 }
 
 void HurtboxBase::_send_next_in_queue()

src/liborng/nodes/hurtbox/hurtbox_base.h

@@ -12,6 +12,7 @@
 #include "character_3d.h"
 
 #include <godot_cpp/classes/collision_shape3d.hpp>
+#include <godot_cpp/classes/shape3d.hpp>
 #include <godot_cpp/classes/area3d.hpp>
 using namespace godot;
 
@@ -23,11 +24,13 @@ class HurtboxBase : public Area3D
 	GDCLASS(HurtboxBase, Area3D);
 	
 public:
+	virtual void _enter_tree() override;
+	virtual void _exit_tree() override;
 	virtual void _ready() override;
 	virtual void _physics_process(double delta) override;
 	
-	void set_collision_shape(CollisionShape3D *cs) { this->collision_shape = cs; }
+	void set_shape(Ref<Shape3D> s) { this->shape = s; }
-	CollisionShape3D *get_collision_shape() const { return this->collision_shape; }
+	Ref<Shape3D> get_shape() const { return this->shape; }
 	
 	void set_lifetime(const float lt) { this->lifetime = lt; }
 	float get_lifetime() const { return this->lifetime; }
@@ -42,6 +45,7 @@ private:
 	Dictionary _create_hit_data_struct() const;
 	void _send_next_in_queue();
 	
+	Ref<Shape3D> shape = nullptr;
 	CollisionShape3D *collision_shape = nullptr;
 	
 	float lifetime = 0.05f;
@@ -55,7 +59,7 @@ private:
 protected:
 	static void _bind_methods()
 	{
-		ADD_GETTER_SETTER_HINTED(HurtboxBase, collision_shape, Variant::OBJECT, PROPERTY_HINT_NODE_TYPE, "CollisionShape3D");
+		ADD_GETTER_SETTER_HINTED(HurtboxBase, shape, Variant::OBJECT, PROPERTY_HINT_RESOURCE_TYPE, "Shape3D");
 		
 		ADD_GETTER_SETTER(HurtboxBase, lifetime, Variant::FLOAT);
 		ADD_GETTER_SETTER(HurtboxBase, hit_points, Variant::INT);

src/liborng/nodes/mesh_editing_library.cpp

@@ -490,15 +490,7 @@ PackedVector3Array MeshEditingLibrary::slice_mesh(MeshInstance3D *original_mesh_
 		std::vector<MeshEditPolygon2D> error_polygons;
 		const Transform3D &plane_inverse_transform = MeshEditingLibrary::project_edges(edges_2d, original_mesh_instance->get_transform(), clip_edges, local_plane);
 		MeshEditingLibrary::build_2d_polygons_from_edges(polygon_set, edges_2d, error_polygons);
-		
+		MeshEditingLibrary::draw_debug_polygons(original_mesh_instance, error_polygons, plane_inverse_transform, Color(1.0f, 0.0f, 0.0f, 1.0f));
-		if (error_polygons.size() > 0)
-		{
-			Ref<Mesh> debug_mesh = MeshEditingLibrary::draw_debug_polygons(error_polygons, plane_inverse_transform, Color(1.0f, 0.0f, 0.0f, 1.0f));
-			MeshInstance3D *mi = memnew(MeshInstance3D);
-			original_mesh_instance->add_child(mi);
-			mi->set_transform(Transform3D());
-			mi->set_mesh(debug_mesh);
-		}
 		
 		MeshSlicePlaneOrientation uv_plane = MeshSlicePlaneOrientation::Z;
 		const Basis &mesh_instance_basis = original_mesh_instance->get_basis();
@@ -880,9 +872,12 @@ const Vector2 MeshEditingLibrary::calculate_planar_uv(const Vector3 &vertex, con
 }
 
 
-Ref<Mesh> MeshEditingLibrary::draw_debug_polygons(const std::vector<MeshEditPolygon2D> &polygons, const Transform3D &plane_transform, const Color &colour)
+void MeshEditingLibrary::draw_debug_polygons(Node3D *mesh_parent_node, const std::vector<MeshEditPolygon2D> &polygons, const Transform3D &plane_transform, const Color &colour)
 {
 #ifdef DEBUG_ENABLED
+	if (polygons.size() <= 0)
+		return;
+	
 	Ref<StandardMaterial3D> material;
 	material.instantiate();
 	material->set_flag(BaseMaterial3D::FLAG_ALBEDO_FROM_VERTEX_COLOR, true);
@@ -912,6 +907,9 @@ Ref<Mesh> MeshEditingLibrary::draw_debug_polygons(const std::vector<MeshEditPoly
 	}
 	im->surface_end();
 	
-	return im;
+	MeshInstance3D *mi = memnew(MeshInstance3D);
+	mesh_parent_node->add_child(mi);
+	mi->set_transform(Transform3D());
+	mi->set_mesh(im);
 #endif // DEBUG_ENABLED
 }

src/liborng/nodes/mesh_editing_library.h

@@ -90,7 +90,7 @@ private:
 	static bool find_next_edge(MeshEditEdge2D &out_next_edge, std::vector<MeshEditEdge2D> &in_edge_set, const MeshEditVert2D &start);
 	static void fix_polygon_winding(MeshEditPolygon2D &polygon);
 	
-	static Ref<Mesh> draw_debug_polygons(const std::vector<MeshEditPolygon2D> &polygons, const Transform3D &plane_transform, const Color &colour);
+	static void draw_debug_polygons(Node3D *mesh_parent_node, const std::vector<MeshEditPolygon2D> &polygons, const Transform3D &plane_transform, const Color &colour);
 	
 	// Godot boilerplate below
 protected:

src/liborng/nodes/player_spawn.cpp

@@ -68,7 +68,7 @@ bool PlayerOrigin::spawn_player(Node3D *parent, Ref<PackedScene> player) const
 			#ifdef DEBUG_ENABLED
 			else
 			{
-				UtilityFunctions::push_error("Empty sublevel in PlayerSpawn object \"", this->tag, "\"");
+				PRINT_ERROR(PLAYERSPAWN_LOG_TAG, "Empty sublevel in PlayerSpawn object \"", this->tag, "\"");
 			}
 			#endif
 		}

src/liborng/nodes/player_spawn.h

@@ -14,6 +14,8 @@
 #include <godot_cpp/classes/packed_scene.hpp>
 using namespace godot;
 
+#define PLAYERSPAWN_LOG_TAG "PlayerSpawn"
+
 #define PLAYER_ORIGIN_TAG	"origin"
 
 

src/liborng/register_types.cpp

@@ -88,9 +88,9 @@ void uninitialize_orng_module(ModuleInitializationLevel p_level) {
 	if (p_level == ModuleInitializationLevel::MODULE_INITIALIZATION_LEVEL_SCENE)
 	{
 		// GDSINGLETON_UNREGISTER_CLASS(_vendor_service_singleton);
-		// GDSINGLETON_UNREGISTER_CLASS(_scene_loader_singleton);
+		GDSINGLETON_UNREGISTER_CLASS(_scene_loader_singleton);
-		// GDSINGLETON_UNREGISTER_CLASS(_save_manager_singleton);
+		GDSINGLETON_UNREGISTER_CLASS(_save_manager_singleton);
-		// GDSINGLETON_UNREGISTER_CLASS(_input_handler_singleton);
+		GDSINGLETON_UNREGISTER_CLASS(_input_handler_singleton);
 	}
 }
 

src/liborng/resources/input_resource.cpp

@@ -55,7 +55,8 @@ void InputResource::update_axes(const float delta)
 		const StringName &signal_1d = prefix + INPUTAXIS_ONE_DIMENSIONAL_SUFFIX + INPUTHANDLER_SUFFIX_SIGNAL;
 		if (this->has_signal(signal_1d))
 		{
-			const float x_axis = axis->get_x_axis();
+			float x_axis = axis->get_x_axis();
+			if (UtilityFunctions::absf(x_axis) < axis_deadzone) x_axis = 0.0f;
 			if (current_input.x != x_axis || current_input.length() > SMALL_NUMBER)
 				this->emit_signal(signal_1d, delta, x_axis);
 			current_input.x = x_axis;
@@ -64,7 +65,9 @@ void InputResource::update_axes(const float delta)
 		const StringName &signal_2d = prefix + INPUTAXIS_TWO_DIMENSIONAL_SUFFIX + INPUTHANDLER_SUFFIX_SIGNAL;
 		if (this->has_signal(signal_2d))
 		{
-			const Vector2 &vector = axis->get_lateral_vector();
+			Vector2 &vector = axis->get_lateral_vector();
+			if (UtilityFunctions::absf(vector.x) < axis_deadzone) vector.x = 0.0f;
+			if (UtilityFunctions::absf(vector.y) < axis_deadzone) vector.y = 0.0f;
 			if ((current_input.x != vector.x || current_input.z != vector.y) || current_input.length() > SMALL_NUMBER)
 				this->emit_signal(signal_2d, delta, vector);
 			current_input.x = vector.x; current_input.z = vector.y;
@@ -73,7 +76,10 @@ void InputResource::update_axes(const float delta)
 		const StringName &signal_3d = prefix + INPUTAXIS_THREE_DIMENSIONAL_SUFFIX + INPUTHANDLER_SUFFIX_SIGNAL;
 		if (this->has_signal(signal_3d))
 		{
-			const Vector3 &vector = axis->get_spherical_vector();
+			Vector3 &vector = axis->get_spherical_vector();
+			if (UtilityFunctions::absf(vector.x) < axis_deadzone) vector.x = 0.0f;
+			if (UtilityFunctions::absf(vector.y) < axis_deadzone) vector.y = 0.0f;
+			if (UtilityFunctions::absf(vector.z) < axis_deadzone) vector.z = 0.0f;
 			if (current_input != vector || current_input.length() > SMALL_NUMBER)
 				this->emit_signal(signal_3d, delta, vector);
 			current_input = vector;