diff options
author | Maks Naumov <maksqwe1@ukr.net> | 2015-02-07 12:10:41 +0200 |
---|---|---|
committer | Maks Naumov <maksqwe1@ukr.net> | 2015-02-08 12:25:12 +0000 |
commit | c2f26d6d0b70e591c74bf6664e30288a2d63dc9a (patch) | |
tree | 243d25d8fc79463ec7a3cc091e3da2574b89c4ff /src/sql/models/qsqltablemodel.cpp | |
parent | 163b8529516ddda24833f635f737863ba98bd688 (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.cpp | 3 |
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; } |