split out DumpList

stepmania/src/smpackage/LanguagesDlg.cpp

@@ -442,6 +442,28 @@ void GetAllMatches( const RString &sRegex, const RString &sString, vector<RStrin
 	}
 }
 
+void DumpList( const vector<RString> &asList, RageFile &file )
+{
+	RString sLine;
+	FOREACH_CONST( RString, asList, s )
+	{
+		if( sLine.size() + s->size() > 100 )
+		{
+			file.PutLine( sLine );
+			sLine = "";
+		}
+
+		if( sLine.size() )
+			sLine += ", ";
+		else
+			sLine += "    ";
+
+		sLine += *s;
+	}
+
+	file.PutLine( sLine );
+}
+
 void LanguagesDlg::OnBnClickedCheckLanguage()
 {
 	RString sTheme = GetCurrentString( m_listThemes );
@@ -521,37 +543,23 @@ void LanguagesDlg::OnBnClickedCheckLanguage()
 	{
 		FOREACH_CONST_Child( &ini1, key )
 		{
-			bool bFirst = true;
+			vector<RString> asList;
-			RString sLastSection;
+			const RString &sSection = key->m_sName;
-			RString sLine;
 			FOREACH_CONST_Attr( key, value )
 			{
-				const RString &sSection = key->m_sName;
 				const RString &sID = value->first;
 				RString sCurrentLanguage;
 				ini2.GetValue( sSection, sID, sCurrentLanguage );
 				if( !sCurrentLanguage.empty() )
 					continue;
 
-				if( bFirst )
+				asList.push_back( sID );
+			}
+			if( !asList.empty() )
 			{
-					bFirst = false;
 				file.PutLine( ssprintf("Not translated in section [%s]:", sSection.c_str()) );
+				DumpList( asList, file );
 			}
-				if( sLine.size() + sID.size() > 100 )
-				{
-					file.PutLine( sLine );
-					sLine = "";
-				}
-				if( sLine.size() )
-					sLine += ", ";
-				else
-					sLine += "    ";
-
-				sLine += sID;
-			}
-			if( sLine.size() )
-				file.PutLine( sLine );
 		}
 	}