always brace nested ifs, or you end up with obscure bugs like this one
This commit is contained in:
@@ -177,12 +177,14 @@ void ScreenPrompt::MenuStart( PlayerNumber pn )
|
|||||||
SOUNDMAN->PlayOnce( THEME->GetPathToS("Common start") );
|
SOUNDMAN->PlayOnce( THEME->GetPathToS("Common start") );
|
||||||
|
|
||||||
if( m_bAnswer )
|
if( m_bAnswer )
|
||||||
|
{
|
||||||
if( m_pOnYes )
|
if( m_pOnYes )
|
||||||
m_pOnYes();
|
m_pOnYes();
|
||||||
else
|
} else {
|
||||||
if( m_pOnNo )
|
if( m_pOnNo )
|
||||||
m_pOnNo();
|
m_pOnNo();
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
void ScreenPrompt::MenuBack( PlayerNumber pn )
|
void ScreenPrompt::MenuBack( PlayerNumber pn )
|
||||||
{
|
{
|
||||||
|
|||||||
Reference in New Issue
Block a user