AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix QJsonValue comparison.

Browse Source 
QJsonValue, while comparing two QJsonObjects, should consult also length
of the objects, because a different than null base pointer doesn't mean
that an object is not empty.

Change-Id: Ibee1849ef9fed15d32f2c8f2aad9b053846e46b7
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
This commit is contained in:
Jędrzej Nowacki 2014-06-26 13:14:25 +02:00
parent 20cf632ad5
commit 77d7348be2
2 changed files with 14 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/corelib/json/qjsonvalue.cpp
View File

@ -613,8 +613,10 @@ bool QJsonValue::operator==(const QJsonValue &other) const
case Object:
if (base == other.base)
return true;
if (!base || !other.base)
return false;
if (!base)
return !other.base->length;
if (!other.base)
return !base->length;
return QJsonObject(d, static_cast<QJsonPrivate::Object *>(base))
== QJsonObject(other.d, static_cast<QJsonPrivate::Object *>(other.base));
}

14
tests/auto/corelib/json/tst_qtjson.cpp
View File

@ -2444,14 +2444,20 @@ void tst_QtJson::objectEquals()
QFETCH(QJsonObject, right);
QFETCH(bool, result);
QVERIFY((left == right) == result);
QVERIFY((right == left) == result);
QCOMPARE(left == right, result);
QCOMPARE(right == left, result);
// invariants checks
QCOMPARE(left, left);
QCOMPARE(right, right);
QVERIFY((left != right) != result);
QVERIFY((right != left) != result);
QCOMPARE(left != right, !result);
QCOMPARE(right != left, !result);
// The same but from QJsonValue perspective
QCOMPARE(QJsonValue(left) == QJsonValue(right), result);
QCOMPARE(QJsonValue(left) != QJsonValue(right), !result);
QCOMPARE(QJsonValue(right) == QJsonValue(left), result);
QCOMPARE(QJsonValue(right) != QJsonValue(left), !result);
}
void tst_QtJson::bom()