silence GCC warnings

Author: brakhane

CMakeLists.txt

@@ -82,10 +82,13 @@ elseif (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
         -Wwrite-strings
         -Winit-self
         -Wreturn-type
-        -Wreorder
-        -Werror=delete-non-virtual-dtor
+        $<$<COMPILE_LANGUAGE:CXX>:-Wreorder>
+        $<$<COMPILE_LANGUAGE:CXX>:-Werror=delete-non-virtual-dtor>
         -Werror
-        #uncomment this to stop the compilation at the first error
+
+		# some GCC versions have bugs triggering this warning
+	    -Wno-stringop-overflow
+		#uncomment this to stop the compilation at the first error
         # -Wfatal-errors
     )
 endif()

Editor/Editor.cpp

@@ -7,7 +7,7 @@
 #include "DummyVisualizer.h"
 
 // some application parameters can be overwritten in the executable by finding the 256 byte long pattern in the first member:
-extern ApplicationExeCustomization exe_customization = {
+ApplicationExeCustomization exe_customization = {
 	"Wicked Editor                                                                                                          ",
 	wi::Color(130, 210, 220, 255),
 	wi::Color(17, 30, 43, 255)

Editor/ModelImporter_GLTF.cpp

@@ -2227,7 +2227,7 @@ void Import_Extension_VRM(LoaderState& state)
 				for (size_t boneGroup_index = 0; boneGroup_index < boneGroups.ArrayLen(); ++boneGroup_index)
 				{
 					const auto& boneGroup = boneGroups.Get(int(boneGroup_index));
-					SpringComponent component;
+					SpringComponent component{};
 
 					if (boneGroup.Has("dragForce"))
 					{
@@ -2966,7 +2966,7 @@ void Import_Extension_VRMC(LoaderState& state)
 					for (size_t joint_index = 0; joint_index < joints.ArrayLen(); ++joint_index)
 					{
 						const auto& joint = joints.Get(int(joint_index));
-						SpringComponent component;
+						SpringComponent component{};
 						//component.colliders = colliderIDs; // for now, we will just use all colliders in the scene for every spring
 
 						if (joint.Has("dragForce"))
@@ -4767,7 +4767,7 @@ void ExportModel_GLTF(const std::string& filename, Scene& scene)
 
 			// Build accessors
 
-			size_t morph_pos_accessor_index;
+			size_t morph_pos_accessor_index = -1;
 			if(buf_d_morph_pos_size > 0)
 			{
 				// Sparse accessor indices
@@ -4812,7 +4812,7 @@ void ExportModel_GLTF(const std::string& filename, Scene& scene)
 				state.gltfModel.accessors.push_back(morph_pos_accessor_builder);
 			}
 
-			size_t morph_norm_accessor_index;
+			size_t morph_norm_accessor_index = -1;
 			if(buf_d_morph_norm_size > 0)
 			{
 				// Sparse accessor indices

Editor/ModelImporter_OBJ.cpp

@@ -2,9 +2,15 @@
 #include "wiScene.h"
 #include "ModelImporter.h"
 
+#ifdef __GNUC__
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
+#endif
 #define TINYOBJLOADER_IMPLEMENTATION
 #include "tiny_obj_loader.h"
-
+#ifdef __GNUC__
+#pragma GCC diagnostic pop
+#endif
 using namespace wi::graphics;
 using namespace wi::scene;
 using namespace wi::ecs;