diff options
author | Jason McDonald <jason.mcdonald@nokia.com> | 2011-05-10 01:33:37 (GMT) |
---|---|---|
committer | Jason McDonald <jason.mcdonald@nokia.com> | 2011-05-10 02:47:54 (GMT) |
commit | 6e3fcd6ff50e6435fd90629ed695196d25312ffc (patch) | |
tree | 39f1062c0b238a40d1992da248863ec8c3b88b05 /tests | |
parent | f859ab10715a3cb69aa3f03574a046c886d34b44 (diff) | |
download | Qt-6e3fcd6ff50e6435fd90629ed695196d25312ffc.zip Qt-6e3fcd6ff50e6435fd90629ed695196d25312ffc.tar.gz Qt-6e3fcd6ff50e6435fd90629ed695196d25312ffc.tar.bz2 |
Remove Q_ASSERT's from QXmlStream autotest
Report fatal errors rather than ignoring them in non-debug builds.
Change-Id: Ieafc58f3603fa953f4a963394039e613dcd442fb
Task-number: QTBUG-17582
Reviewed-by: Rohan McGovern
Diffstat (limited to 'tests')
-rw-r--r-- | tests/auto/qxmlstream/qc14n.h | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/tests/auto/qxmlstream/qc14n.h b/tests/auto/qxmlstream/qc14n.h index 99432f3..c7d3a7d 100644 --- a/tests/auto/qxmlstream/qc14n.h +++ b/tests/auto/qxmlstream/qc14n.h @@ -71,12 +71,14 @@ bool QC14N::isEqual(QIODevice *const firstDocument, QString *const message) { qDebug() << Q_FUNC_INFO; - Q_ASSERT_X(firstDocument, Q_FUNC_INFO, - "A valid QIODevice pointer must be supplied"); - Q_ASSERT_X(secondDocument, Q_FUNC_INFO, - "A valid QIODevice pointer must be supplied"); - Q_ASSERT_X(firstDocument->isReadable(), Q_FUNC_INFO, "The device must be readable."); - Q_ASSERT_X(secondDocument->isReadable(), Q_FUNC_INFO, "The device must be readable."); + if (!firstDocument) + qFatal("%s: A valid firstDocument QIODevice pointer must be supplied", Q_FUNC_INFO); + if (!secondDocument) + qFatal("%s: A valid secondDocument QIODevice pointer must be supplied", Q_FUNC_INFO); + if (!firstDocument->isReadable()) + qFatal("%s: The firstDocument device must be readable.", Q_FUNC_INFO); + if (!secondDocument->isReadable()) + qFatal("%s: The secondDocument device must be readable.", Q_FUNC_INFO); QXmlStreamReader r1(firstDocument); QXmlStreamReader r2(secondDocument); |