Skip to content

Commit cbe1a0e

Browse files
committed
gameui: Use range-for loops to simplify message dailog handler
1 parent d17dab2 commit cbe1a0e

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

gameui/BasePanel.cpp

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3730,19 +3730,18 @@ void CMessageDialogHandler::ShowMessageDialog( int nType, vgui::Panel *pOwner )
37303730

37313731
void CMessageDialogHandler::CloseAllMessageDialogs()
37323732
{
3733-
for ( int i = 0; i < MAX_MESSAGE_DIALOGS; ++i )
3733+
for ( auto &dialog : m_hMessageDialogs )
37343734
{
3735-
CMessageDialog *pDlg = m_hMessageDialogs[i];
3736-
if ( pDlg )
3735+
if ( dialog )
37373736
{
37383737
vgui::surface()->RestrictPaintToSinglePanel(NULL);
37393738
if ( vgui_message_dialog_modal.GetBool() )
37403739
{
37413740
vgui::input()->ReleaseAppModalSurface();
37423741
}
37433742

3744-
pDlg->Close();
3745-
m_hMessageDialogs[i] = NULL;
3743+
dialog->Close();
3744+
dialog = nullptr;
37463745
}
37473746
}
37483747
}
@@ -3844,11 +3843,11 @@ void CMessageDialogHandler::ActivateMessageDialog( int nStackIdx )
38443843

38453844
void CMessageDialogHandler::PositionDialogs( int wide, int tall )
38463845
{
3847-
for ( int i = 0; i < MAX_MESSAGE_DIALOGS; ++i )
3846+
for ( auto &dialog : m_hMessageDialogs )
38483847
{
3849-
if ( m_hMessageDialogs[i].Get() )
3848+
if ( dialog.Get() )
38503849
{
3851-
PositionDialog( m_hMessageDialogs[i], wide, tall );
3850+
PositionDialog( dialog, wide, tall );
38523851
}
38533852
}
38543853
}

0 commit comments

Comments
 (0)