Replaces return"" by return QString()
authorLuís Pereira <luis.artur.pereira@gmail.com>
Tue, 27 Dec 2011 22:33:16 +0000 (22:33 +0000)
committerLuís Pereira <luis.artur.pereira@gmail.com>
Tue, 27 Dec 2011 22:33:16 +0000 (22:33 +0000)
It's faster and cleaner.
        QString().isEmpty();            // returns true
        QString().isNull();             // returns true
So replacing "" by QString() will not brake existing code.

Signed-off-by: Luís Pereira <luis.artur.pereira@gmail.com>
razorsettings.cpp

index c546192..ba9b4f1 100644 (file)
@@ -120,7 +120,7 @@ QString findFile(const QString& fileName, bool onlyGlobal = false)
             return file;
     }
 
-    return "";
+    return QString();
 }
 
 
@@ -287,7 +287,7 @@ QString RazorTheme::qss(const QString& module) const
         return d->loadQss(path);
 
     qWarning() << "QSS file cannot be found in any location:" << QString("%1/%2.qss").arg(d->mThemeName, module);
-    return "";
+    return QString();
 }
 
 
@@ -300,14 +300,14 @@ QString RazorThemePrivate::loadQss(const QString& qssFile) const
     if (! f.open(QIODevice::ReadOnly | QIODevice::Text))
     {
         qWarning() << "Theme: Canot open file for reading:" << qssFile;
-        return "";
+        return QString();
     }
 
     QString qss = f.readAll();
     f.close();
 
     if (qss.isEmpty())
-        return "";
+        return QString();
 
     // handle relative paths
     QString qssDir = QFileInfo(qssFile).canonicalPath();
@@ -326,7 +326,7 @@ QString RazorTheme::desktopBackground(int screen) const
     QString wallpapperCfgFileName = findFile(QString("themes/%1/wallpapper.cfg").arg(d->mThemeName));
 
     if (wallpapperCfgFileName.isEmpty())
-        return "";
+        return QString();
 
     QSettings s(wallpapperCfgFileName, QSettings::IniFormat);
     QString themeDir = QFileInfo(wallpapperCfgFileName).absolutePath();