From 4c47079d367d44cf18b91965c351a5e9d834b343 Mon Sep 17 00:00:00 2001 From: Aaron Kennedy Date: Thu, 29 Apr 2010 18:53:38 +1000 Subject: Simplify QDeclarativeGuard logic --- src/declarative/qml/qdeclarativedata_p.h | 10 ++++++---- src/declarative/qml/qdeclarativeengine.cpp | 12 ++++-------- 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/src/declarative/qml/qdeclarativedata_p.h b/src/declarative/qml/qdeclarativedata_p.h index 4a56536..e916273 100644 --- a/src/declarative/qml/qdeclarativedata_p.h +++ b/src/declarative/qml/qdeclarativedata_p.h @@ -152,11 +152,11 @@ public: template void QDeclarativeGuard::addGuard() { - if (QObjectPrivate::get(o)->wasDeleted) { - if (prev) remGuard(); + Q_ASSERT(!prev); + + if (QObjectPrivate::get(o)->wasDeleted) return; - } - + QDeclarativeData *data = QDeclarativeData::get(o, true); next = data->guards; if (next) reinterpret_cast *>(next)->prev = &next; @@ -167,6 +167,8 @@ void QDeclarativeGuard::addGuard() template void QDeclarativeGuard::remGuard() { + Q_ASSERT(prev); + if (next) reinterpret_cast *>(next)->prev = prev; *prev = next; next = 0; diff --git a/src/declarative/qml/qdeclarativeengine.cpp b/src/declarative/qml/qdeclarativeengine.cpp index 19d4b57..8992d50 100644 --- a/src/declarative/qml/qdeclarativeengine.cpp +++ b/src/declarative/qml/qdeclarativeengine.cpp @@ -870,14 +870,10 @@ void QDeclarativeData::destroyed(QObject *object) if (ownContext && context) context->destroy(); - QDeclarativeGuard *guard = guards; - while (guard) { - QDeclarativeGuard *g = guard; - guard = guard->next; - g->o = 0; - g->prev = 0; - g->next = 0; - g->objectDestroyed(object); + while (guards) { + QDeclarativeGuard *guard = guards; + *guard = (QObject *)0; + guard->objectDestroyed(object); } if (scriptValue) -- cgit v0.12