Don't rebuild the vector of human players for each row.

2006-09-01 07:46:37 +00:00
parent 763738cafa
commit 37dc530c0a
1 changed files with 4 additions and 4 deletions
@@ -241,15 +241,15 @@ void ScreenOptions::InitMenu( const vector<OptionRowHandler*> &vHands )
 void ScreenOptions::RestartOptions()
 {
 	m_exprRowPositionTransformFunction.ClearCache();
-
+	vector<PlayerNumber> vpns;
+	FOREACH_HumanPlayer( p )
+		vpns.push_back( p );
+	
 	for( unsigned r=0; r<m_pRows.size(); r++ )		// foreach row
 	{
 		OptionRow *pRow = m_pRows[r];
 		pRow->Reload();

-		vector<PlayerNumber> vpns;
-		FOREACH_HumanPlayer( p )
-			vpns.push_back( p );
 		this->ImportOptions( r, vpns );
 	
 		// HACK: Process disabled players, too, to hide inactive underlines.