mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tdf#99973 - Crash when changing Locale Setting in options

Browse Source 
regression from commit 0f672545 "clang-tidy modernize-loop-convert in
toolkit to uui", probably because the list is being modified while the
loop is executing

Change-Id: I18cd21001aee132e06d577c469aed5eb46a16a7f
Reviewed-on: https://gerrit.libreoffice.org/25368
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
This commit is contained in:
Noel Grandin 2016-05-23 15:16:01 +02:00 committed by Noel Grandin
parent 2a16ad7e4a
commit 3a077c0c3f
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
unotools/source/config/options.cxx
View File

@ -68,9 +68,8 @@ void ConfigurationBroadcaster::NotifyListeners( sal_uInt32 nHint )
nHint |= m_nBlockedHint; nHint |= m_nBlockedHint;
m_nBlockedHint = 0; m_nBlockedHint = 0;
if ( mpList ) { if ( mpList ) {
for (ConfigurationListener* n : *mpList) { for ( size_t n = 0; n < mpList->size(); n++ )
n->ConfigurationChanged( this, nHint ); (*mpList)[ n ]->ConfigurationChanged( this, nHint );
}
} }
} }
} }