lovecraft/qmltermwidget
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Prevent a c++11 range-loop might detach Qt container

Browse Source 
Just make it const.
This commit is contained in:
Luís Pereira
2020-01-14 19:02:06 +00:00
committed by Alf Gaida
parent 887efa8151
commit ae71042e0d
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/ColorScheme.cpp
+2 -2
View File
@@ -730,7 +730,7 @@ QList<QString> ColorSchemeManager::listKDE3ColorSchemes()
QStringList filters;
filters << QLatin1String("*.schema");
dir.setNameFilters(filters);
QStringList list = dir.entryList(filters);
const QStringList list = dir.entryList(filters);
for (const QString &i : list)
ret << dname + QLatin1Char('/') + i;
}
@@ -750,7 +750,7 @@ QList<QString> ColorSchemeManager::listColorSchemes()
QStringList filters;
filters << QLatin1String("*.colorscheme");
dir.setNameFilters(filters);
QStringList list = dir.entryList(filters);
const QStringList list = dir.entryList(filters);
for (const QString &i : list)
ret << dname + QLatin1Char('/') + i;
}
lib/qtermwidget.cpp
+1 -1
View File
@@ -276,7 +276,7 @@ void QTermWidget::init(int startnow)
m_translator = new QTranslator(this);
for (const QString& dir : dirs) {
for (const QString& dir : qAsConst(dirs)) {
//qDebug() << "Trying to load translation file from dir" << dir;
if (m_translator->load(QLocale::system(), QLatin1String("qtermwidget"), QLatin1String(QLatin1String("_")), dir)) {
qApp->installTranslator(m_translator);