cleanup
This commit is contained in:
@@ -176,7 +176,7 @@ void ScreenSelect::FinalizeChoices()
|
|||||||
FOREACH_PlayerNumber( p )
|
FOREACH_PlayerNumber( p )
|
||||||
if( GAMESTATE->IsHumanPlayer(p) )
|
if( GAMESTATE->IsHumanPlayer(p) )
|
||||||
{
|
{
|
||||||
const int sel = GetSelectionIndex( (PlayerNumber)p );
|
const int sel = GetSelectionIndex( p );
|
||||||
|
|
||||||
if( m_aModeChoices[sel].m_pStyle )
|
if( m_aModeChoices[sel].m_pStyle )
|
||||||
GAMESTATE->m_pCurStyle = m_aModeChoices[sel].m_pStyle;
|
GAMESTATE->m_pCurStyle = m_aModeChoices[sel].m_pStyle;
|
||||||
|
|||||||
@@ -602,19 +602,17 @@ void ScreenSelectMaster::MenuStart( PlayerNumber pn )
|
|||||||
if( SHARED_PREVIEW_AND_CURSOR || GetCurrentPage() == PAGE_2 )
|
if( SHARED_PREVIEW_AND_CURSOR || GetCurrentPage() == PAGE_2 )
|
||||||
{
|
{
|
||||||
/* Only one player has to pick. Choose this for all the other players, too. */
|
/* Only one player has to pick. Choose this for all the other players, too. */
|
||||||
FOREACH_PlayerNumber( p )
|
FOREACH_HumanPlayer( p )
|
||||||
if( GAMESTATE->IsHumanPlayer(p) )
|
|
||||||
{
|
{
|
||||||
ASSERT( !m_bChosen[p] );
|
ASSERT( !m_bChosen[p] );
|
||||||
fSecs = max( fSecs, DoMenuStart( (PlayerNumber)p ) );
|
fSecs = max( fSecs, DoMenuStart(p) );
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
fSecs = max( fSecs, DoMenuStart(pn) );
|
fSecs = max( fSecs, DoMenuStart(pn) );
|
||||||
// check to see if everyone has chosen
|
// check to see if everyone has chosen
|
||||||
FOREACH_PlayerNumber( p )
|
FOREACH_HumanPlayer( p )
|
||||||
if( GAMESTATE->IsHumanPlayer((PlayerNumber)p) )
|
|
||||||
bAllDone &= m_bChosen[p];
|
bAllDone &= m_bChosen[p];
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user