diff --git a/stepmania/src/ScreenOptions.cpp b/stepmania/src/ScreenOptions.cpp index 6408925060..2685b69121 100644 --- a/stepmania/src/ScreenOptions.cpp +++ b/stepmania/src/ScreenOptions.cpp @@ -138,8 +138,9 @@ void ScreenOptions::Init( InputMode im, OptionRowData OptionRows[], int iNumOpti for( int r=0; r &vbSelected = Row.m_vbSelected[p]; - vbSelected.resize( Row.m_RowDef.choices.size() ); + vector &vbSelected = row.m_vbSelected[p]; + vbSelected.resize( row.m_RowDef.choices.size() ); for( unsigned j=0; jWarn( "Options menu \"%s\" row %i has no selection", m_sName.c_str(), i ); + LOG->Warn( "Options menu \"%s\" row index %i has no selection", m_sName.c_str(), r ); row.m_vbSelected[p][0] = true; } @@ -245,18 +246,15 @@ void ScreenOptions::Init( InputMode im, OptionRowData OptionRows[], int iNumOpti for( unsigned r=0; r & textItems = row.m_textItems; const OptionRowData &optline = m_Rows[r]->m_RowDef; - unsigned c; - m_framePage.AddChild( &row.m_sprBullet ); m_framePage.AddChild( &row.m_textTitle ); float fX = ITEMS_START_X; // indent 70 pixels - for( c=0; cSetX( fX ); // init underlines - FOREACH_PlayerNumber( p ) + FOREACH_HumanPlayer( p ) { - if( !GAMESTATE->IsHumanPlayer(p) ) - continue; - OptionsCursor *ul = new OptionsCursor; row.m_Underline[p].push_back( ul ); ul->Load( (PlayerNumber)p, true ); @@ -294,12 +289,12 @@ void ScreenOptions::Init( InputMode im, OptionRowData OptionRows[], int iNumOpti { row.m_bRowIsLong = true; for( unsigned j=0; jLoad( (PlayerNumber)p, true ); + ul->Load( p, true ); int iWidth, iX, iY; - GetWidthXY( (PlayerNumber) p, r, c, iWidth, iX, iY ); + GetWidthXY( p, r, 0, iWidth, iX, iY ); ul->SetX( float(iX) ); ul->SetWidth( float(iWidth) ); } @@ -375,14 +370,12 @@ void ScreenOptions::Init( InputMode im, OptionRowData OptionRows[], int iNumOpti InitOptionsText(); // add explanation here so it appears on top + FOREACH_PlayerNumber( p ) { - FOREACH_PlayerNumber( p ) - { - m_textExplanation[p].LoadFromFont( THEME->GetPathToF("ScreenOptions explanation") ); - m_textExplanation[p].SetZoom( EXPLANATION_ZOOM ); - m_textExplanation[p].SetShadowLength( 0 ); - m_framePage.AddChild( &m_textExplanation[p] ); - } + m_textExplanation[p].LoadFromFont( THEME->GetPathToF("ScreenOptions explanation") ); + m_textExplanation[p].SetZoom( EXPLANATION_ZOOM ); + m_textExplanation[p].SetShadowLength( 0 ); + m_framePage.AddChild( &m_textExplanation[p] ); } /* Hack: if m_CurStyle is set, we're probably in the player or song options menu, so