Remove Q_ASSERT's from ecmascript test.
authorJason McDonald <jason.mcdonald@nokia.com>
Wed, 4 May 2011 02:26:59 +0000 (12:26 +1000)
committerRohan McGovern <rohan.mcgovern@nokia.com>
Wed, 18 May 2011 00:59:11 +0000 (10:59 +1000)
These Q_ASSERT's were redundant as the test functions that call these
functions already verify that "value == 0".

Change-Id: Ic23e539c5814153d67fea8156ade0783f91d7b83
Task-number: QTBUG-17582
Reviewed-by: Rohan McGovern
(cherry picked from commit 6f791f8ad0b7ef023d91c1609ce076443f335c13)

tests/auto/declarative/qdeclarativeecmascript/tst_qdeclarativeecmascript.cpp

index d9cd825..a408000 100644 (file)
@@ -2272,16 +2272,12 @@ public:
     ~CppOwnershipReturnValue() { delete value; }
 
     Q_INVOKABLE QObject *create() {
-        Q_ASSERT(value == 0);
-
         value = new QObject;
         QDeclarativeEngine::setObjectOwnership(value, QDeclarativeEngine::CppOwnership);
         return value;
     }
 
     Q_INVOKABLE MyQmlObject *createQmlObject() {
-        Q_ASSERT(value == 0);
-
         MyQmlObject *rv = new MyQmlObject;
         value = rv;
         return rv;