Fix warnings

pioneerspacesim · Sep 8, 2023 · ce30223 · ce30223
1 parent f801b07
commit ce30223
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/core/StringUtils.h b/src/core/StringUtils.h
@@ -128,7 +128,7 @@ struct SplitString {
 		using reference = std::string_view;
 
 		// "end" iterator
-		iter() : m_str(), m_parent(nullptr) {};
+		iter() : m_parent(nullptr), m_str() {};
 		// "live" iterator
 		iter(SplitString *parent) :
 			m_parent(parent),

diff --git a/src/pigui/PerfInfo.cpp b/src/pigui/PerfInfo.cpp
@@ -114,6 +114,7 @@ PerfInfo::CounterInfo &PerfInfo::GetCounter(CounterType ct)
 	case COUNTER_FPS: return m_fpsCounter;
 	case COUNTER_PHYS: return m_physCounter;
 	case COUNTER_PIGUI: return m_piguiCounter;
+	default: assert(0);
 	}
 }
 

diff --git a/src/pigui/PiGui.cpp b/src/pigui/PiGui.cpp
@@ -584,7 +584,7 @@ void Instance::BakeFont(PiFont &font)
 
 	// Set the ImGui font name for debugging purposes
 	std::string name = fmt::format("{}:{}", font.name(), font.pixelsize());
-	strncpy(config.Name, name.c_str(), 40);
+	strncpy(config.Name, name.c_str(), 39);
 
 	// The main face of the font should go first in the list, because:
 	//