Use const references with the foreach iterator
authorLuís Pereira <luis.artur.pereira@gmail.com>
Fri, 22 Apr 2016 12:14:42 +0000 (13:14 +0100)
committerLuís Pereira <luis.artur.pereira@gmail.com>
Fri, 22 Apr 2016 17:20:33 +0000 (18:20 +0100)
We should use const references whenever possible.

lxqtplugininfo.cpp
lxqtsettings.cpp
lxqttranslator.cpp

index 243efb2..1f3ed78 100644 (file)
@@ -105,11 +105,11 @@ PluginInfoList PluginInfo::search(const QStringList& desktopFilesDirs, const QSt
     QList<PluginInfo> res;
     QSet<QString> processed;
 
-    foreach (QString desktopFilesDir, desktopFilesDirs)
+    foreach (const QString &desktopFilesDir, desktopFilesDirs)
     {
         QDir dir(desktopFilesDir);
         QFileInfoList files = dir.entryInfoList(QStringList(nameFilter), QDir::Files | QDir::Readable);
-        foreach (QFileInfo file, files)
+        foreach (const QFileInfo &file, files)
         {
             if (processed.contains(file.fileName()))
                 continue;
index 1a43cbc..21a4b1d 100644 (file)
@@ -423,7 +423,7 @@ QString LXQtThemeData::findTheme(const QString &themeName)
     if (!paths.contains(fallback))
         paths << fallback;
 
-    foreach(QString path, paths)
+    foreach(const QString &path, paths)
     {
         QDir dir(QString("%1/lxqt/themes/%2").arg(path, themeName));
         if (dir.isReadable())
@@ -585,12 +585,12 @@ QList<LXQtTheme> LXQtTheme::allThemes()
     paths << XdgDirs::dataHome(false);
     paths << XdgDirs::dataDirs();
 
-    foreach(QString path, paths)
+    foreach(const QString &path, paths)
     {
         QDir dir(QString("%1/lxqt/themes").arg(path));
         QFileInfoList dirs = dir.entryInfoList(QDir::AllDirs | QDir::NoDotAndDotDot);
 
-        foreach(QFileInfo dir, dirs)
+        foreach(const QFileInfo &dir, dirs)
         {
             if (!processed.contains(dir.fileName()) &&
                  QDir(dir.absoluteFilePath()).exists("lxqt-panel.qss"))
index 841f84e..4fc67c0 100644 (file)
@@ -89,7 +89,7 @@ bool translate(const QString &name, const QString &owner)
     QTranslator *appTranslator = new QTranslator(qApp);
 
     QStringList *paths = getSearchPaths();
-    foreach(QString path, *paths)
+    foreach(const QString &path, *paths)
     {
         QStringList subPaths;
 
@@ -103,7 +103,7 @@ bool translate(const QString &name, const QString &owner)
             subPaths << path;
         }
 
-        foreach(QString p, subPaths)
+        foreach(const QString &p, subPaths)
         {
             if (appTranslator->load(name + "_" + locale, p))
             {