Fixes (#17)
authorLuís Pereira <luis.artur.pereira@gmail.com>
Thu, 4 May 2017 21:30:50 +0000 (22:30 +0100)
committerAlf Gaida <agaida@siduction.org>
Thu, 4 May 2017 21:30:50 +0000 (23:30 +0200)
* Puts const references in foreach loops

We don't need a copy, a const reference is enough and much faster.

* Use QString::leftRef() and QString::midRef()

It's faster.

cpustat.cpp
memstat.cpp
netstat.cpp

index 6017d6e..f275af1 100644 (file)
@@ -62,7 +62,7 @@ void CpuStatPrivate::updateSources()
 {
     mSources.clear();
 
-    foreach (QString row, readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts))
+    foreach (const QString &row, readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts))
     {
         QStringList tokens = row.split(QChar(' '), QString::SkipEmptyParts);
         if( (tokens.size() < 5)
@@ -76,15 +76,15 @@ void CpuStatPrivate::updateSources()
 
     bool ok;
 
-    foreach (QString range, readAllFile("/sys/devices/system/cpu/online").split(QChar(','), QString::SkipEmptyParts))
+    foreach (const QString &range, readAllFile("/sys/devices/system/cpu/online").split(QChar(','), QString::SkipEmptyParts))
     {
         int dash = range.indexOf('-');
         if (dash != -1)
         {
-            uint min = range.left(dash).toUInt(&ok);
+            uint min = range.leftRef(dash).toUInt(&ok);
             if (ok)
             {
-                uint max = range.mid(dash + 1).toUInt(&ok);
+                uint max = range.midRef(dash + 1).toUInt(&ok);
                 if (ok)
                     for (uint number = min; number <= max; ++number)
                         addSource(QString("cpu%1").arg(number));
@@ -128,7 +128,7 @@ void CpuStatPrivate::timeout()
     if ( (mMonitoring == CpuStat::LoadOnly)
       || (mMonitoring == CpuStat::LoadAndFrequency) )
     {
-        foreach (QString row, readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts))
+        foreach (const QString &row, readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts))
         {
             if (!row.startsWith("cpu"))
                 continue;
index 074fbc8..bdca252 100644 (file)
@@ -53,7 +53,7 @@ void MemStatPrivate::timeout()
     qulonglong swapTotal = 0;
     qulonglong swapFree = 0;
 
-    foreach (QString row, readAllFile("/proc/meminfo").split(QChar('\n'), QString::SkipEmptyParts))
+    foreach (const QString &row, readAllFile("/proc/meminfo").split(QChar('\n'), QString::SkipEmptyParts))
     {
         QStringList tokens = row.split(QChar(' '), QString::SkipEmptyParts);
         if (tokens.size() != 3)
index e4450a1..970dda2 100644 (file)
@@ -42,7 +42,7 @@ NetStatPrivate::NetStatPrivate(NetStat *parent)
 
     rows.erase(rows.begin(), rows.begin() + 2);
 
-    foreach (QString row, rows)
+    foreach (const QString &row, rows)
     {
         QStringList tokens = row.split(QChar(':'), QString::SkipEmptyParts);
         if (tokens.size() != 2)
@@ -74,7 +74,7 @@ void NetStatPrivate::timeout()
 
     rows.erase(rows.begin(), rows.begin() + 2);
 
-    foreach (QString row, rows)
+    foreach (const QString &row, rows)
     {
         QStringList tokens = row.split(QChar(':'), QString::SkipEmptyParts);
         if (tokens.size() != 2)