From 37dc530c0af21ff1e9ed658e02a16f93230abb3f Mon Sep 17 00:00:00 2001
From: Steve Checkoway <stevecheckoway@users.sourceforge.net>
Date: Fri, 1 Sep 2006 07:46:37 +0000
Subject: [PATCH] Don't rebuild the vector of human players for each row.

---
 stepmania/src/ScreenOptions.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/stepmania/src/ScreenOptions.cpp b/stepmania/src/ScreenOptions.cpp
index e4d4b7f197..3adfbef3ef 100644
--- a/stepmania/src/ScreenOptions.cpp
+++ b/stepmania/src/ScreenOptions.cpp
@@ -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.