Drop Qt foreach.
authorLuís Pereira <luis.artur.pereira@gmail.com>
Thu, 28 Dec 2017 11:42:23 +0000 (11:42 +0000)
committerAlf Gaida <agaida@siduction.org>
Sun, 31 Dec 2017 21:40:06 +0000 (22:40 +0100)
Replaced with the ranged-for loop.
Using QT_NO_FOREACH to enforce it.
Qt 5.7.1 required. We use qAsConst().

CMakeLists.txt
cpustat.cpp
memstat.cpp
netstat.cpp

index 4d162ba..79d89d6 100644 (file)
@@ -8,7 +8,7 @@ set(PATCH_VERSION 0)
 set(SYSSTAT_VERSION "${MAJOR_VERSION}.${MINOR_VERSION}.${PATCH_VERSION}")
 
 set(LXQTBT_MINIMUM_VERSION "0.4.0")
-set(QT_MINIMUM_VERSION "5.4.2")
+set(QT_MINIMUM_VERSION "5.7.1")
 
 # additional cmake files
 list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake")
@@ -93,6 +93,7 @@ target_compile_definitions(${SYSSTAT_LIBRARY_NAME}
         "MINOR_VERSION=${MINOR_VERSION}"
         "PATCH_VERSION=${PATCH_VERSION}"
         "SYSSTAT_LIBRARY"
+        "QT_NO_FOREACH"
 )
 
 target_include_directories(${SYSSTAT_LIBRARY_NAME}
index f275af1..fb2920a 100644 (file)
@@ -62,7 +62,8 @@ void CpuStatPrivate::updateSources()
 {
     mSources.clear();
 
-    foreach (const QString &row, readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts))
+    const QStringList rows = readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts);
+    for (const QString &row : rows)
     {
         QStringList tokens = row.split(QChar(' '), QString::SkipEmptyParts);
         if( (tokens.size() < 5)
@@ -76,7 +77,8 @@ void CpuStatPrivate::updateSources()
 
     bool ok;
 
-    foreach (const QString &range, readAllFile("/sys/devices/system/cpu/online").split(QChar(','), QString::SkipEmptyParts))
+    const QStringList ranges = readAllFile("/sys/devices/system/cpu/online").split(QChar(','), QString::SkipEmptyParts);
+    for (const QString &range : ranges)
     {
         int dash = range.indexOf('-');
         if (dash != -1)
@@ -128,7 +130,8 @@ void CpuStatPrivate::timeout()
     if ( (mMonitoring == CpuStat::LoadOnly)
       || (mMonitoring == CpuStat::LoadAndFrequency) )
     {
-        foreach (const QString &row, readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts))
+        const QStringList rows = readAllFile("/proc/stat").split(QChar('\n'), QString::SkipEmptyParts);
+        for (const QString &row : rows)
         {
             if (!row.startsWith("cpu"))
                 continue;
index bdca252..38eff68 100644 (file)
@@ -53,7 +53,8 @@ void MemStatPrivate::timeout()
     qulonglong swapTotal = 0;
     qulonglong swapFree = 0;
 
-    foreach (const QString &row, readAllFile("/proc/meminfo").split(QChar('\n'), QString::SkipEmptyParts))
+    const QStringList rows = readAllFile("/proc/meminfo").split(QChar('\n'), QString::SkipEmptyParts);
+    for (const QString &row : rows)
     {
         QStringList tokens = row.split(QChar(' '), QString::SkipEmptyParts);
         if (tokens.size() != 3)
index 970dda2..f582d02 100644 (file)
@@ -42,7 +42,7 @@ NetStatPrivate::NetStatPrivate(NetStat *parent)
 
     rows.erase(rows.begin(), rows.begin() + 2);
 
-    foreach (const QString &row, rows)
+    for (const QString &row : qAsConst(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 (const QString &row, rows)
+    for (const QString &row : qAsConst(rows))
     {
         QStringList tokens = row.split(QChar(':'), QString::SkipEmptyParts);
         if (tokens.size() != 2)