From de8a9dee5aa7c8db3b390c9a19f65c919d201c3c Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Fri, 25 Jul 2014 23:17:55 +0200 Subject: QPalette: add move constructor As with many other implicitly shared classes, efficient move semantics requires setting the d-pointer to nullptr, which then needs to be checked for in the dtor and the copy assignment operator. Change-Id: I654d181a1dfdd9a16e2f9fb96b57475cdd0b4561 Reviewed-by: J-P Nurmi Reviewed-by: Olivier Goffart --- tests/auto/gui/kernel/qpalette/tst_qpalette.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'tests') diff --git a/tests/auto/gui/kernel/qpalette/tst_qpalette.cpp b/tests/auto/gui/kernel/qpalette/tst_qpalette.cpp index b2c0ff979c..662e35dfaf 100644 --- a/tests/auto/gui/kernel/qpalette/tst_qpalette.cpp +++ b/tests/auto/gui/kernel/qpalette/tst_qpalette.cpp @@ -106,6 +106,14 @@ void tst_QPalette::moveSemantics() src = control; // check moved-from 'src' can still be assigned to (doesn't crash) QVERIFY(src.isCopyOf(dst)); QVERIFY(src.isCopyOf(control)); + QPalette dst2(qMove(src)); // move construction + QVERIFY(!src.isCopyOf(dst)); + QVERIFY(!src.isCopyOf(dst2)); + QVERIFY(!src.isCopyOf(control)); + QCOMPARE(dst2, control); + QVERIFY(dst2.isCopyOf(dst)); + QVERIFY(dst2.isCopyOf(control)); + // check moved-from 'src' can still be destroyed (doesn't crash) #else QSKIP("Compiler doesn't support C++11 move semantics"); #endif -- cgit v1.2.3