Minor correction for gcc complaining about signed-unsigned comparison

Drewol · Mar 21, 2021 · c5acb48 · c5acb48
1 parent 60f358e
commit c5acb48
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/Main/src/SettingsScreen.cpp b/Main/src/SettingsScreen.cpp
@@ -105,13 +105,16 @@ class SettingsPage
 		void Load()
 		{
 			String str = g_gameConfig.GetString(m_key);
-			m_len = static_cast<int>(str.length());
 
-			if (m_len >= m_buffer.size())
+			if (str.length() >= m_buffer.size())
 			{
 				Logf("Config key=%d cropped due to being too long (%d)", Logger::Severity::Error, static_cast<int>(m_key), m_len);
 				m_len = static_cast<int>(m_buffer.size() - 1);
 			}
+			else
+			{
+				m_len = static_cast<int>(str.length());
+			}
 
 			std::memcpy(m_buffer.data(), str.data(), m_len + 1);
 		}