diff options
author | Christian Ehrlicher <ch.ehrlicher@gmx.de> | 2022-12-21 17:35:35 +0100 |
---|---|---|
committer | Christian Ehrlicher <ch.ehrlicher@gmx.de> | 2022-12-22 19:26:50 +0100 |
commit | 5419faebdef749f72e718a5a21b04efa2f6e3160 (patch) | |
tree | 16b81d15eb7402a309504a0095d65504ad2a285c /src/sql/kernel/qsqlrecord.cpp | |
parent | 8b0818235d512c9085ce0fd0ea2490f1653a6c36 (diff) |
SQL: misc cleanup in QSqlRecord/QSqlField
Cleanup an unused function in QSqlRecordPrivate and use member
initializers for the ctors in QSqlRecord/QSqlField
Change-Id: I7d585e70c83373b3092e9f1425233af42b8c379d
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
Diffstat (limited to 'src/sql/kernel/qsqlrecord.cpp')
-rw-r--r-- | src/sql/kernel/qsqlrecord.cpp | 44 |
1 files changed, 14 insertions, 30 deletions
diff --git a/src/sql/kernel/qsqlrecord.cpp b/src/sql/kernel/qsqlrecord.cpp index 74f42b2e48..a3c6a9debf 100644 --- a/src/sql/kernel/qsqlrecord.cpp +++ b/src/sql/kernel/qsqlrecord.cpp @@ -15,36 +15,21 @@ QT_BEGIN_NAMESPACE class QSqlRecordPrivate { public: - QSqlRecordPrivate(); - QSqlRecordPrivate(const QSqlRecordPrivate &other); + QSqlRecordPrivate() = default; + QSqlRecordPrivate(const QSqlRecordPrivate &other) + : fields(other.fields) + { + } - inline bool contains(int index) { return index >= 0 && index < fields.size(); } - QString createField(int index, const QString &prefix) const; + inline bool contains(qsizetype index) const + { + return index >= 0 && index < fields.size(); + } QList<QSqlField> fields; - QAtomicInt ref; + QAtomicInt ref{1}; }; -QSqlRecordPrivate::QSqlRecordPrivate() : ref(1) -{ -} - -QSqlRecordPrivate::QSqlRecordPrivate(const QSqlRecordPrivate &other): fields(other.fields), ref(1) -{ -} - -/*! \internal - Just for compat -*/ -QString QSqlRecordPrivate::createField(int index, const QString &prefix) const -{ - QString f; - if (!prefix.isEmpty()) - f = prefix + u'.'; - f += fields.at(index).name(); - return f; -} - /*! \class QSqlRecord \brief The QSqlRecord class encapsulates a database record. @@ -86,8 +71,8 @@ QString QSqlRecordPrivate::createField(int index, const QString &prefix) const */ QSqlRecord::QSqlRecord() + : d(new QSqlRecordPrivate) { - d = new QSqlRecordPrivate(); } /*! @@ -98,8 +83,8 @@ QSqlRecord::QSqlRecord() */ QSqlRecord::QSqlRecord(const QSqlRecord& other) + : d(other.d) { - d = other.d; d->ref.ref(); } @@ -337,9 +322,8 @@ bool QSqlRecord::contains(const QString& name) const void QSqlRecord::clearValues() { detach(); - int count = d->fields.size(); - for (int i = 0; i < count; ++i) - d->fields[i].clear(); + for (QSqlField &f : d->fields) + f.clear(); } /*! |