diff options
author | Mark Brand <mabrand@mabrand.nl> | 2013-04-03 21:44:30 +0200 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-04-04 13:13:37 +0200 |
commit | 0bdc86d9ef7be8e27598d245c6ca8026f08aff12 (patch) | |
tree | 53d6556f439ebfdd92a5dc24d4dac61daf3f9869 /src/sql/drivers/ibase/qsql_ibase_p.h | |
parent | 708d39fa31368f80152440e94e4a361d024b19c1 (diff) |
QSqlDriver: use Q_DECLARE_PUBLIC/Q_DECLARE_PRIVATE
406c8ef6e67da introduced deriving the private SQL driver classes from
QSqlDriverPrivate. However, the drivers continued to keep their own
pointer to the private class, even though QObject provides the same
pointer. Worse yet, the private class is allocated too late and not
even passed to QSqlDriver. The result is that QSqlDriver allocates
a separate instance of QSqlDriverPrivate. This is likely to cause
all kinds of chaos.
The private class needs to be allocated in time pass it to QSqlDriver
which passes it on to QObject.
This commit covers the the base class and drivers:
ibase
mysql
odbc
psql
sqlite
tds
Fixes for the remaining drivers will follow.
Change-Id: Id8e7ec4205b0ca6cd00bd022c9cd24f137089245
Reviewed-by: Andy Shaw <andy.shaw@digia.com>
Diffstat (limited to 'src/sql/drivers/ibase/qsql_ibase_p.h')
-rw-r--r-- | src/sql/drivers/ibase/qsql_ibase_p.h | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/sql/drivers/ibase/qsql_ibase_p.h b/src/sql/drivers/ibase/qsql_ibase_p.h index b2560ca17c..ab9edfd1a5 100644 --- a/src/sql/drivers/ibase/qsql_ibase_p.h +++ b/src/sql/drivers/ibase/qsql_ibase_p.h @@ -64,9 +64,9 @@ class QIBaseDriver; class QIBaseDriver : public QSqlDriver { - Q_OBJECT - friend class QIBaseDriverPrivate; friend class QIBaseResultPrivate; + Q_DECLARE_PRIVATE(QIBaseDriver) + Q_OBJECT public: explicit QIBaseDriver(QObject *parent = 0); explicit QIBaseDriver(isc_db_handle connection, QObject *parent = 0); @@ -104,9 +104,6 @@ public: private Q_SLOTS: void qHandleEventNotification(void* updatedResultBuffer); - -private: - QIBaseDriverPrivate* d; }; QT_END_NAMESPACE |