From f6011ec52836bff8009739ff3b75c2daf65b3c4c Mon Sep 17 00:00:00 2001 From: Mark Brand Date: Thu, 4 Apr 2013 10:37:04 +0200 Subject: QSQLITE2Driver: use Q_DECLARE_PUBLIC/Q_DECLARE_PRIVATE Follow-up to 0bdc86d9ef7be8 Change-Id: I0a480b5780eb22b1c22e8a47b3a13ab1cd97b934 Reviewed-by: Andy Shaw --- src/sql/drivers/sqlite2/qsql_sqlite2.cpp | 20 +++++++++++++------- src/sql/drivers/sqlite2/qsql_sqlite2_p.h | 6 ++---- 2 files changed, 15 insertions(+), 11 deletions(-) (limited to 'src') diff --git a/src/sql/drivers/sqlite2/qsql_sqlite2.cpp b/src/sql/drivers/sqlite2/qsql_sqlite2.cpp index 39ad122822..7e56d5cc17 100644 --- a/src/sql/drivers/sqlite2/qsql_sqlite2.cpp +++ b/src/sql/drivers/sqlite2/qsql_sqlite2.cpp @@ -281,8 +281,8 @@ QSQLite2Result::QSQLite2Result(const QSQLite2Driver* db) : QSqlCachedResult(db) { d = new QSQLite2ResultPrivate(this); - d->access = db->d->access; - d->utf8 = db->d->utf8; + d->access = db->d_func()->access; + d->utf8 = db->d_func()->utf8; } QSQLite2Result::~QSQLite2Result() @@ -376,16 +376,15 @@ QVariant QSQLite2Result::handle() const ///////////////////////////////////////////////////////// -QSQLite2Driver::QSQLite2Driver(QObject * parent) - : QSqlDriver(parent) +QSQLite2Driver::QSQLite2Driver(QObject *parent) + : QSqlDriver(*new QSQLite2DriverPrivate, parent) { - d = new QSQLite2DriverPrivate(); } QSQLite2Driver::QSQLite2Driver(sqlite *connection, QObject *parent) - : QSqlDriver(parent) + : QSqlDriver(*new QSQLite2DriverPrivate, parent) { - d = new QSQLite2DriverPrivate(); + Q_D(QSQLite2Driver); d->access = connection; setOpen(true); setOpenError(false); @@ -398,6 +397,7 @@ QSQLite2Driver::~QSQLite2Driver() bool QSQLite2Driver::hasFeature(DriverFeature f) const { + Q_D(const QSQLite2Driver); switch (f) { case Transactions: case SimpleLocking: @@ -415,6 +415,7 @@ bool QSQLite2Driver::hasFeature(DriverFeature f) const */ bool QSQLite2Driver::open(const QString & db, const QString &, const QString &, const QString &, int, const QString &) { + Q_D(QSQLite2Driver); if (isOpen()) close(); @@ -441,6 +442,7 @@ bool QSQLite2Driver::open(const QString & db, const QString &, const QString &, void QSQLite2Driver::close() { + Q_D(QSQLite2Driver); if (isOpen()) { sqlite_close(d->access); d->access = 0; @@ -456,6 +458,7 @@ QSqlResult *QSQLite2Driver::createResult() const bool QSQLite2Driver::beginTransaction() { + Q_D(QSQLite2Driver); if (!isOpen() || isOpenError()) return false; @@ -473,6 +476,7 @@ bool QSQLite2Driver::beginTransaction() bool QSQLite2Driver::commitTransaction() { + Q_D(QSQLite2Driver); if (!isOpen() || isOpenError()) return false; @@ -490,6 +494,7 @@ bool QSQLite2Driver::commitTransaction() bool QSQLite2Driver::rollbackTransaction() { + Q_D(QSQLite2Driver); if (!isOpen() || isOpenError()) return false; @@ -586,6 +591,7 @@ QSqlRecord QSQLite2Driver::record(const QString &tbl) const QVariant QSQLite2Driver::handle() const { + Q_D(const QSQLite2Driver); return QVariant::fromValue(d->access); } diff --git a/src/sql/drivers/sqlite2/qsql_sqlite2_p.h b/src/sql/drivers/sqlite2/qsql_sqlite2_p.h index 7a075210ae..ae38a662ac 100644 --- a/src/sql/drivers/sqlite2/qsql_sqlite2_p.h +++ b/src/sql/drivers/sqlite2/qsql_sqlite2_p.h @@ -71,8 +71,9 @@ class QSQLite2Driver; class QSQLite2Driver : public QSqlDriver { - Q_OBJECT friend class QSQLite2Result; + Q_DECLARE_PRIVATE(QSQLite2Driver) + Q_OBJECT public: explicit QSQLite2Driver(QObject *parent = 0); explicit QSQLite2Driver(sqlite *connection, QObject *parent = 0); @@ -100,9 +101,6 @@ public: QSqlIndex primaryIndex(const QString &table) const; QVariant handle() const; QString escapeIdentifier(const QString &identifier, IdentifierType) const; - -private: - QSQLite2DriverPrivate* d; }; QT_END_NAMESPACE -- cgit v1.2.3