summaryrefslogtreecommitdiffstats
path: root/src/sql/models/qsqltablemodel.cpp
diff options
context:
space:
mode:
authorMaks Naumov <maksqwe1@ukr.net>2015-02-07 12:10:41 +0200
committerMaks Naumov <maksqwe1@ukr.net>2015-02-08 12:25:12 +0000
commitc2f26d6d0b70e591c74bf6664e30288a2d63dc9a (patch)
tree243d25d8fc79463ec7a3cc091e3da2574b89c4ff /src/sql/models/qsqltablemodel.cpp
parent163b8529516ddda24833f635f737863ba98bd688 (diff)
Use prefix instead of postfix for iterators
The postfix increment(decrement) creates a temp copy of *this before the modification and then returns that copy. It's needed only when using the old iterator and then incrementing it. Change-Id: I7f6702de78f5f987cec3556047e76049b4ee063a Reviewed-by: Konstantin Ritt <ritt.ks@gmail.com> Reviewed-by: Marc Mutz <marc.mutz@kdab.com>
Diffstat (limited to 'src/sql/models/qsqltablemodel.cpp')
-rw-r--r--src/sql/models/qsqltablemodel.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/sql/models/qsqltablemodel.cpp b/src/sql/models/qsqltablemodel.cpp
index e0e700cd01..faecad3771 100644
--- a/src/sql/models/qsqltablemodel.cpp
+++ b/src/sql/models/qsqltablemodel.cpp
@@ -501,9 +501,10 @@ bool QSqlTableModel::isDirty() const
Q_D(const QSqlTableModel);
QSqlTableModelPrivate::CacheMap::ConstIterator i = d->cache.constBegin();
const QSqlTableModelPrivate::CacheMap::ConstIterator e = d->cache.constEnd();
- for (; i != e; i++)
+ for (; i != e; ++i) {
if (!i.value().submitted())
return true;
+ }
return false;
}