Remove repeated invocations of end() in iterations.

Assign end() to a variable. Where possible,
use foreach() to iterate over lists or map values or use
Qt convenience APIs.

Change-Id: Ie6a6f56d1b0ddb419d06a7ccf4be2f8b32c029b4
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/src/activeqt/shared/qaxtypes.cpp b/src/activeqt/shared/qaxtypes.cpp
index 1697686..6eba133 100644
--- a/src/activeqt/shared/qaxtypes.cpp
+++ b/src/activeqt/shared/qaxtypes.cpp
@@ -1246,9 +1246,8 @@ QVariant VARIANTToQVariant(const VARIANT &arg, const QByteArray &typeName, uint
         } else if (proptype == QVariant::StringList && var.type() == QVariant::List) {
             bool allStrings = true;
             QStringList strings;
-            const QList<QVariant> list(var.toList());
-            for (QList<QVariant>::ConstIterator it(list.begin()); it != list.end(); ++it) {
-                QVariant variant = *it;
+            const QVariantList list(var.toList());
+            foreach (const QVariant &variant, list) {
                 if (variant.canConvert(QVariant::String))
                     strings << variant.toString();
                 else