kdeconnect-kde/plasmoid/declarativeplugin
Jean Vincent c864267f04 Replace Q_FOREACH with C++11 range-for
Summary:
The use of Q_FOREACH is advised against (https://doc.qt.io/qt-5/qtglobal.html#Q_FOREACH) since Qt 5.7 and will eventually be removed from Qt.

I replaced all occurrences with the range-for loop introduced in C++11 (except for the one in daemon.cpp in deviceIdByName which might have a bug / typo in it).

I added const to the container or casted it with qAsConst when appropriate to avoid unnecessary copies.

(This is my first submission. I did all the unit tests, and they all passed but I don't know how to show it here.)

Reviewers: #kde_connect, nicolasfella, apol

Reviewed By: #kde_connect, nicolasfella, apol

Subscribers: albertvaka, apol, nicolasfella

Tags: #kde_connect

Differential Revision: https://phabricator.kde.org/D6724
2017-07-21 09:57:19 +02:00
..
qml Show the announcedName in the qml app 2016-08-21 19:38:15 +02:00
CMakeLists.txt Show the announcedName in the qml app 2016-08-21 19:38:15 +02:00
kdeconnectdeclarativeplugin.cpp Add remotekeyboard plugin 2017-01-23 09:08:27 +01:00
kdeconnectdeclarativeplugin.h Clean up use of virtual and override keywords 2016-06-20 19:22:29 +02:00
objectfactory.h Clean up use of virtual and override keywords 2016-06-20 19:22:29 +02:00
processrunner.cpp Clazy says all this QStringLiteral and QLatin1String were missing/wrong 2016-11-26 16:21:29 +01:00
processrunner.h Clean up use of virtual and override keywords 2016-06-20 19:22:29 +02:00
qmldir Show the announcedName in the qml app 2016-08-21 19:38:15 +02:00
responsewaiter.cpp Replace Q_FOREACH with C++11 range-for 2017-07-21 09:57:19 +02:00
responsewaiter.h Clean up use of virtual and override keywords 2016-06-20 19:22:29 +02:00