From: Friedemann Kleint Date: Thu, 17 Nov 2011 14:52:53 +0000 (+0100) Subject: qtbase tests: Fix some warnings X-Git-Tag: qt-v5.0.0-alpha1~2617 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e8b4c49efddb87f4195c74668b5026119eccea56;p=profile%2Fivi%2Fqtbase.git qtbase tests: Fix some warnings - Fix warnings about QAtomicPointer/Int usage - Fix some gcc 4.6 warnings about assigned/unused variables Change-Id: Ib4dbf9110f0dad93ad48e97278310f05fad3a82a Reviewed-by: Rohan McGovern --- diff --git a/tests/auto/corelib/animation/qanimationgroup/tst_qanimationgroup.cpp b/tests/auto/corelib/animation/qanimationgroup/tst_qanimationgroup.cpp index b92794e..736b4d2 100644 --- a/tests/auto/corelib/animation/qanimationgroup/tst_qanimationgroup.cpp +++ b/tests/auto/corelib/animation/qanimationgroup/tst_qanimationgroup.cpp @@ -301,14 +301,14 @@ void tst_QAnimationGroup::setParentAutoAdd() void tst_QAnimationGroup::beginNestedGroup() { - QAnimationGroup *subGroup; QAnimationGroup *parent = new QParallelAnimationGroup(); for (int i = 0; i < 10; ++i) { - if (i & 1) - subGroup = new QParallelAnimationGroup(parent); - else - subGroup = new QSequentialAnimationGroup(parent); + if (i & 1) { + new QParallelAnimationGroup(parent); + } else { + new QSequentialAnimationGroup(parent); + } QCOMPARE(parent->animationCount(), 1); QAnimationGroup *child = static_cast(parent->animationAt(0)); diff --git a/tests/auto/corelib/concurrent/qfuture/tst_qfuture.cpp b/tests/auto/corelib/concurrent/qfuture/tst_qfuture.cpp index 964b834..b959dcb 100644 --- a/tests/auto/corelib/concurrent/qfuture/tst_qfuture.cpp +++ b/tests/auto/corelib/concurrent/qfuture/tst_qfuture.cpp @@ -95,7 +95,6 @@ void tst_QFuture::resultStore() { ResultStore store; - ResultIteratorBase it = store.begin(); QVERIFY(store.begin() == store.end()); QVERIFY(store.resultAt(0) == store.end()); QVERIFY(store.resultAt(1) == store.end()); diff --git a/tests/auto/corelib/concurrent/qtconcurrentiteratekernel/tst_qtconcurrentiteratekernel.cpp b/tests/auto/corelib/concurrent/qtconcurrentiteratekernel/tst_qtconcurrentiteratekernel.cpp index ed71410..96ba71b 100644 --- a/tests/auto/corelib/concurrent/qtconcurrentiteratekernel/tst_qtconcurrentiteratekernel.cpp +++ b/tests/auto/corelib/concurrent/qtconcurrentiteratekernel/tst_qtconcurrentiteratekernel.cpp @@ -233,7 +233,8 @@ public: bool shouldThrottleThread() { - return (iterations > 100 && iterations < 200); + const int load = iterations.load(); + return (load > 100 && load < 200); } bool throttling; }; diff --git a/tests/auto/corelib/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp b/tests/auto/corelib/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp index 1c60621..a1dd1c7 100644 --- a/tests/auto/corelib/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp +++ b/tests/auto/corelib/concurrent/qtconcurrentmap/tst_qtconcurrentmap.cpp @@ -1285,6 +1285,7 @@ void tst_QtConcurrentMap::mappedReduced() // test Q_DECLARE_OPERATORS_FOR_FLAGS QtConcurrent::ReduceOptions opt = (QtConcurrent::UnorderedReduce | QtConcurrent::SequentialReduce); + QVERIFY(opt); // functor-functor { diff --git a/tests/auto/corelib/concurrent/qtconcurrentresultstore/tst_qtconcurrentresultstore.cpp b/tests/auto/corelib/concurrent/qtconcurrentresultstore/tst_qtconcurrentresultstore.cpp index 1728be6..f5b3f29 100644 --- a/tests/auto/corelib/concurrent/qtconcurrentresultstore/tst_qtconcurrentresultstore.cpp +++ b/tests/auto/corelib/concurrent/qtconcurrentresultstore/tst_qtconcurrentresultstore.cpp @@ -88,7 +88,6 @@ void tst_QtConcurrentResultStore::iterators() { { ResultStore store; - ResultIteratorBase it = store.begin(); QVERIFY(store.begin() == store.end()); QVERIFY(store.resultAt(0) == store.end()); QVERIFY(store.resultAt(1) == store.end()); diff --git a/tests/auto/corelib/kernel/qmetaobjectbuilder/tst_qmetaobjectbuilder.cpp b/tests/auto/corelib/kernel/qmetaobjectbuilder/tst_qmetaobjectbuilder.cpp index 719b20e..7ad020c 100644 --- a/tests/auto/corelib/kernel/qmetaobjectbuilder/tst_qmetaobjectbuilder.cpp +++ b/tests/auto/corelib/kernel/qmetaobjectbuilder/tst_qmetaobjectbuilder.cpp @@ -1023,12 +1023,12 @@ void tst_QMetaObjectBuilder::removeNotifySignal() { QMetaObjectBuilder builder; - QMetaMethodBuilder method1 = builder.addSignal("foo(const QString&, int)"); - QMetaMethodBuilder method2 = builder.addSignal("bar(QString)"); + builder.addSignal("foo(const QString&, int)"); + QMetaMethodBuilder method = builder.addSignal("bar(QString)"); // Setup property QMetaPropertyBuilder prop = builder.addProperty("prop", "const QString &"); - prop.setNotifySignal(method2); + prop.setNotifySignal(method); QVERIFY(prop.hasNotifySignal()); QCOMPARE(prop.notifySignal().index(), 1); diff --git a/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp b/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp index a1c943e..90c537b 100644 --- a/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp +++ b/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp @@ -129,10 +129,8 @@ void tst_QSocketNotifier::unexpectedDisconnection() NATIVESOCKETENGINE readEnd1; readEnd1.initialize(QAbstractSocket::TcpSocket); - bool b = readEnd1.connectToHost(server.serverAddress(), server.serverPort()); + readEnd1.connectToHost(server.serverAddress(), server.serverPort()); QVERIFY(readEnd1.waitForWrite()); -// while (!b && readEnd1.state() != QAbstractSocket::ConnectedState) -// b = readEnd1.connectToHost(server.serverAddress(), server.serverPort()); QVERIFY(readEnd1.state() == QAbstractSocket::ConnectedState); QVERIFY(server.waitForNewConnection()); QTcpSocket *writeEnd1 = server.nextPendingConnection(); @@ -140,10 +138,8 @@ void tst_QSocketNotifier::unexpectedDisconnection() NATIVESOCKETENGINE readEnd2; readEnd2.initialize(QAbstractSocket::TcpSocket); - b = readEnd2.connectToHost(server.serverAddress(), server.serverPort()); + readEnd2.connectToHost(server.serverAddress(), server.serverPort()); QVERIFY(readEnd2.waitForWrite()); -// while (!b) -// b = readEnd2.connectToHost(server.serverAddress(), server.serverPort()); QVERIFY(readEnd2.state() == QAbstractSocket::ConnectedState); QVERIFY(server.waitForNewConnection()); QTcpSocket *writeEnd2 = server.nextPendingConnection(); diff --git a/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp b/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp index 73330b1..f70c380 100644 --- a/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp +++ b/tests/auto/corelib/kernel/qtimer/tst_qtimer.cpp @@ -430,7 +430,6 @@ public: m_timer.stop(); - QTime now = QTime::currentTime(); int elapsed = m_startedTime.elapsed(); if (elapsed < m_interval / 2) { diff --git a/tests/auto/corelib/thread/qmutex/tst_qmutex.cpp b/tests/auto/corelib/thread/qmutex/tst_qmutex.cpp index cc36af8..82b0c98 100644 --- a/tests/auto/corelib/thread/qmutex/tst_qmutex.cpp +++ b/tests/auto/corelib/thread/qmutex/tst_qmutex.cpp @@ -576,28 +576,28 @@ public: quint64 i = 0; while (t.elapsed() < one_minute) { i++; - uint nb = (i * 9 + lockCount * 13) % threadCount; + uint nb = (i * 9 + lockCount.load() * 13) % threadCount; QMutexLocker locker(&mutex[nb]); - if (sentinel[nb]) errorCount.ref(); + if (sentinel[nb].load()) errorCount.ref(); if (sentinel[nb].fetchAndAddRelaxed(5)) errorCount.ref(); if (!sentinel[nb].testAndSetRelaxed(5, 0)) errorCount.ref(); - if (sentinel[nb]) errorCount.ref(); + if (sentinel[nb].load()) errorCount.ref(); lockCount.ref(); - nb = (nb * 17 + i * 5 + lockCount * 3) % threadCount; + nb = (nb * 17 + i * 5 + lockCount.load() * 3) % threadCount; if (mutex[nb].tryLock()) { - if (sentinel[nb]) errorCount.ref(); + if (sentinel[nb].load()) errorCount.ref(); if (sentinel[nb].fetchAndAddRelaxed(16)) errorCount.ref(); if (!sentinel[nb].testAndSetRelaxed(16, 0)) errorCount.ref(); - if (sentinel[nb]) errorCount.ref(); + if (sentinel[nb].load()) errorCount.ref(); lockCount.ref(); mutex[nb].unlock(); } - nb = (nb * 15 + i * 47 + lockCount * 31) % threadCount; + nb = (nb * 15 + i * 47 + lockCount.load() * 31) % threadCount; if (mutex[nb].tryLock(2)) { - if (sentinel[nb]) errorCount.ref(); + if (sentinel[nb].load()) errorCount.ref(); if (sentinel[nb].fetchAndAddRelaxed(53)) errorCount.ref(); if (!sentinel[nb].testAndSetRelaxed(53, 0)) errorCount.ref(); - if (sentinel[nb]) errorCount.ref(); + if (sentinel[nb].load()) errorCount.ref(); lockCount.ref(); mutex[nb].unlock(); } diff --git a/tests/auto/corelib/tools/qstring/tst_qstring.cpp b/tests/auto/corelib/tools/qstring/tst_qstring.cpp index 5d29b9c..c9cadad 100644 --- a/tests/auto/corelib/tools/qstring/tst_qstring.cpp +++ b/tests/auto/corelib/tools/qstring/tst_qstring.cpp @@ -4541,17 +4541,16 @@ void tst_QString::nanAndInf() QVERIFY(ok); QVERIFY(d == -INFINITY); - long l; - l = QString("INF").toLong(&ok); + QString("INF").toLong(&ok); QVERIFY(!ok); - l = QString("INF").toLong(&ok, 36); + QString("INF").toLong(&ok, 36); QVERIFY(ok); - l = QString("INF0").toLong(&ok, 36); + QString("INF0").toLong(&ok, 36); QVERIFY(ok); - l = QString("0INF0").toLong(&ok, 36); + QString("0INF0").toLong(&ok, 36); QVERIFY(ok); } diff --git a/tests/auto/dbus/qdbusconnection/tst_qdbusconnection.cpp b/tests/auto/dbus/qdbusconnection/tst_qdbusconnection.cpp index 6490bfe..f8f632d 100644 --- a/tests/auto/dbus/qdbusconnection/tst_qdbusconnection.cpp +++ b/tests/auto/dbus/qdbusconnection/tst_qdbusconnection.cpp @@ -1107,7 +1107,7 @@ class VirtualObject: public QDBusVirtualObject public: VirtualObject() :success(true) {} - QString introspect(const QString &path) const + QString introspect(const QString & /* path */) const { return QString(); } diff --git a/tests/auto/dbus/qdbusthreading/tst_qdbusthreading.cpp b/tests/auto/dbus/qdbusthreading/tst_qdbusthreading.cpp index e37e4c3..32ede6c 100644 --- a/tests/auto/dbus/qdbusthreading/tst_qdbusthreading.cpp +++ b/tests/auto/dbus/qdbusthreading/tst_qdbusthreading.cpp @@ -211,8 +211,8 @@ tst_QDBusThreading::tst_QDBusThreading() void tst_QDBusThreading::joinThreads() { - threadJoin.acquire(threadJoinCount); - threadJoinCount = 0; + threadJoin.acquire(threadJoinCount.load()); + threadJoinCount.store(0); } bool tst_QDBusThreading::waitForSignal(QObject *obj, const char *signal, int delay) diff --git a/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp b/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp index 69dd30b..85ff3b2 100644 --- a/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp +++ b/tests/auto/gui/image/qimagereader/tst_qimagereader.cpp @@ -718,7 +718,7 @@ void tst_QImageReader::sizeBeforeFormat() { QImageReader reader(prefix + fileName); - QSize size = reader.size(); + reader.size(); formatB = reader.format(); } diff --git a/tests/auto/gui/kernel/qshortcut/tst_qshortcut.cpp b/tests/auto/gui/kernel/qshortcut/tst_qshortcut.cpp index d761b19..674fa93 100644 --- a/tests/auto/gui/kernel/qshortcut/tst_qshortcut.cpp +++ b/tests/auto/gui/kernel/qshortcut/tst_qshortcut.cpp @@ -891,8 +891,8 @@ void tst_QShortcut::ambiguousItems() pb1.show(); // Must be show for QShortcutMap::correctSubWindow to trigger pb2.show(); - QShortcut *cut1 = setupShortcut(&pb1, "shortcut1-pb1", TriggerSlot1, "M"); - QShortcut *cut2 = setupShortcut(&pb1, "shortcut2-pb2", TriggerSlot2, "M"); + setupShortcut(&pb1, "shortcut1-pb1", TriggerSlot1, "M"); + setupShortcut(&pb1, "shortcut2-pb2", TriggerSlot2, "M"); currentResult = NoResult; sendKeyEvents( Qt::Key_M, 'm' ); @@ -910,7 +910,6 @@ void tst_QShortcut::ambiguousItems() QCOMPARE( ambigResult, Slot1Triggered ); clearAllShortcuts(); - cut1 = 0; cut2 = 0; } @@ -933,8 +932,8 @@ void tst_QShortcut::unicodeCompare() QKeySequence ks1("Ctrl+M"); // Unicode QKeySequence ks2(Qt::CTRL+Qt::Key_M); // non-Unicode - QShortcut *cut1 = setupShortcut(&pb1, "shortcut1-pb1", TriggerSlot1, ks1); - QShortcut *cut2 = setupShortcut(&pb1, "shortcut2-pb2", TriggerSlot2, ks2); + setupShortcut(&pb1, "shortcut1-pb1", TriggerSlot1, ks1); + setupShortcut(&pb1, "shortcut2-pb2", TriggerSlot2, ks2); currentResult = NoResult; sendKeyEvents( Qt::CTRL+Qt::Key_M, 0 ); @@ -945,7 +944,6 @@ void tst_QShortcut::unicodeCompare() QVERIFY( !(ks1 != ks2) ); clearAllShortcuts(); - cut1 = 0; cut2 = 0; } // ------------------------------------------------------------------ diff --git a/tests/auto/gui/text/qtextdocumentfragment/tst_qtextdocumentfragment.cpp b/tests/auto/gui/text/qtextdocumentfragment/tst_qtextdocumentfragment.cpp index c18d81a..51e659d 100644 --- a/tests/auto/gui/text/qtextdocumentfragment/tst_qtextdocumentfragment.cpp +++ b/tests/auto/gui/text/qtextdocumentfragment/tst_qtextdocumentfragment.cpp @@ -3274,7 +3274,6 @@ void tst_QTextDocumentFragment::html_hrMargins() cursor.movePosition(QTextCursor::NextBlock); block = cursor.block(); - QTextBlock hr = block; QVERIFY(qMax(first.blockFormat().bottomMargin(), block.blockFormat().topMargin()) > 0); cursor.movePosition(QTextCursor::NextBlock); diff --git a/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp b/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp index f1c2f6e..42aa300 100644 --- a/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp +++ b/tests/auto/gui/text/qtextlayout/tst_qtextlayout.cpp @@ -1188,7 +1188,7 @@ void tst_QTextLayout::capitalization_allUpperCase() font.setCapitalization(QFont::AllUppercase); QTextLayout layout("Test", font); layout.beginLayout(); - QTextLine line = layout.createLine(); + layout.createLine(); layout.endLayout(); QTextEngine *engine = layout.engine(); @@ -1203,7 +1203,7 @@ void tst_QTextLayout::capitalization_allLowerCase() font.setCapitalization(QFont::AllLowercase); QTextLayout layout("Test", font); layout.beginLayout(); - QTextLine line = layout.createLine(); + layout.createLine(); layout.endLayout(); QTextEngine *engine = layout.engine(); @@ -1218,7 +1218,7 @@ void tst_QTextLayout::capitalization_smallCaps() font.setCapitalization(QFont::SmallCaps); QTextLayout layout("Test", font); layout.beginLayout(); - QTextLine line = layout.createLine(); + layout.createLine(); layout.endLayout(); QTextEngine *engine = layout.engine(); @@ -1234,7 +1234,7 @@ void tst_QTextLayout::capitalization_capitalize() font.setCapitalization(QFont::Capitalize); QTextLayout layout("hello\tworld", font); layout.beginLayout(); - QTextLine line = layout.createLine(); + layout.createLine(); layout.endLayout(); QTextEngine *engine = layout.engine(); diff --git a/tests/auto/integrationtests/collections/tst_collections.cpp b/tests/auto/integrationtests/collections/tst_collections.cpp index 3901778..e815ab7 100644 --- a/tests/auto/integrationtests/collections/tst_collections.cpp +++ b/tests/auto/integrationtests/collections/tst_collections.cpp @@ -2283,11 +2283,10 @@ int CacheFoo::counter = 0; void tst_Collections::cache() { { - CacheFoo* cf; QCache cache(120); int i; for (i = 0; i < 30; i++) { - cf = cache.object(10); + cache.object(10); cache.insert(i, new CacheFoo(i), i); } @@ -3009,6 +3008,7 @@ void instantiateContainer() constIt = constContainer.end(); container.constEnd(); + Q_UNUSED(constIt) #endif container.clear(); container.contains(value); @@ -3032,6 +3032,7 @@ void instantiateMutableIterationContainer() typename ContainerType::iterator it; it = container.begin(); it = container.end(); + Q_UNUSED(it) #endif // QSet lacks count(T). diff --git a/tests/auto/v8/v8test.cpp b/tests/auto/v8/v8test.cpp index d9a71dd..3792679 100644 --- a/tests/auto/v8/v8test.cpp +++ b/tests/auto/v8/v8test.cpp @@ -82,7 +82,7 @@ bool v8test_externalteardown() Persistent context = Context::New(); Context::Scope context_scope(context); - Local str = String::NewExternal(new MyStringResource); + String::NewExternal(new MyStringResource); Local ft = FunctionTemplate::New(); ft->InstanceTemplate()->SetHasExternalResource(true); diff --git a/tests/auto/widgets/dialogs/qfiledialog/tst_qfiledialog.cpp b/tests/auto/widgets/dialogs/qfiledialog/tst_qfiledialog.cpp index 483841e..bdc7609 100644 --- a/tests/auto/widgets/dialogs/qfiledialog/tst_qfiledialog.cpp +++ b/tests/auto/widgets/dialogs/qfiledialog/tst_qfiledialog.cpp @@ -517,11 +517,9 @@ void tst_QFiledialog::completer() if (!fullPath.endsWith(QLatin1Char('/'))) fullPath.append(QLatin1Char('/')); fullPath.append(input); - bool inputStartsWithRootPath = false; if (input.startsWith(QDir::rootPath())) { fullPath = input; input.clear(); - inputStartsWithRootPath = true; } QFileInfo fi(fullPath); diff --git a/tests/auto/widgets/dialogs/qmessagebox/tst_qmessagebox.cpp b/tests/auto/widgets/dialogs/qmessagebox/tst_qmessagebox.cpp index 3bf618b..f195017 100644 --- a/tests/auto/widgets/dialogs/qmessagebox/tst_qmessagebox.cpp +++ b/tests/auto/widgets/dialogs/qmessagebox/tst_qmessagebox.cpp @@ -522,6 +522,7 @@ void tst_QMessageBox::testSymbols() button = QMessageBox::Default; button = QMessageBox::Escape; button = QMessageBox::FlagMask; + QVERIFY(button); const QString text = QStringLiteral("Foo"); mb1.setText(text); diff --git a/tests/auto/widgets/graphicsview/qgraphicsproxywidget/tst_qgraphicsproxywidget.cpp b/tests/auto/widgets/graphicsview/qgraphicsproxywidget/tst_qgraphicsproxywidget.cpp index f943daf..3f782d0 100644 --- a/tests/auto/widgets/graphicsview/qgraphicsproxywidget/tst_qgraphicsproxywidget.cpp +++ b/tests/auto/widgets/graphicsview/qgraphicsproxywidget/tst_qgraphicsproxywidget.cpp @@ -578,7 +578,6 @@ void tst_QGraphicsProxyWidget::eventFilter() break; } case QEvent::Resize: { - QSize oldSize = widget->size(); QSize newSize = QSize(100, 100); if (fromObject) { widget->resize(newSize); @@ -590,7 +589,6 @@ void tst_QGraphicsProxyWidget::eventFilter() break; } case QEvent::Move: { - QPoint oldPoint = widget->pos(); QPoint newPoint = QPoint(100, 100); if (fromObject) { widget->move(newPoint); @@ -2651,26 +2649,25 @@ void tst_QGraphicsProxyWidget::childPos_data() QTest::addColumn("moveCombo"); QTest::addColumn("comboPos"); QTest::addColumn("proxyPos"); - QTest::addColumn("menuPos"); - - QTest::newRow("0") << true << QPoint() << QPointF() << QPointF(); - QTest::newRow("1") << true << QPoint(10, 0) << QPointF(10, 0) << QPointF(); - QTest::newRow("2") << true << QPoint(100, 0) << QPointF(100, 0) << QPointF(); - QTest::newRow("3") << true << QPoint(1000, 0) << QPointF(1000, 0) << QPointF(); - QTest::newRow("4") << true << QPoint(10000, 0) << QPointF(10000, 0) << QPointF(); - QTest::newRow("5") << true << QPoint(-10000, 0) << QPointF(-10000, 0) << QPointF(); - QTest::newRow("6") << true << QPoint(-1000, 0) << QPointF(-1000, 0) << QPointF(); - QTest::newRow("7") << true << QPoint(-100, 0) << QPointF(-100, 0) << QPointF(); - QTest::newRow("8") << true << QPoint(-10, 0) << QPointF(-10, 0) << QPointF(); - QTest::newRow("0-") << false << QPoint() << QPointF() << QPointF(); - QTest::newRow("1-") << false << QPoint(10, 0) << QPointF(10, 0) << QPointF(); - QTest::newRow("2-") << false << QPoint(100, 0) << QPointF(100, 0) << QPointF(); - QTest::newRow("3-") << false << QPoint(1000, 0) << QPointF(1000, 0) << QPointF(); - QTest::newRow("4-") << false << QPoint(10000, 0) << QPointF(10000, 0) << QPointF(); - QTest::newRow("5-") << false << QPoint(-10000, 0) << QPointF(-10000, 0) << QPointF(); - QTest::newRow("6-") << false << QPoint(-1000, 0) << QPointF(-1000, 0) << QPointF(); - QTest::newRow("7-") << false << QPoint(-100, 0) << QPointF(-100, 0) << QPointF(); - QTest::newRow("8-") << false << QPoint(-10, 0) << QPointF(-10, 0) << QPointF(); + + QTest::newRow("0") << true << QPoint() << QPointF(); + QTest::newRow("1") << true << QPoint(10, 0) << QPointF(10, 0); + QTest::newRow("2") << true << QPoint(100, 0) << QPointF(100, 0); + QTest::newRow("3") << true << QPoint(1000, 0) << QPointF(1000, 0); + QTest::newRow("4") << true << QPoint(10000, 0) << QPointF(10000, 0); + QTest::newRow("5") << true << QPoint(-10000, 0) << QPointF(-10000, 0); + QTest::newRow("6") << true << QPoint(-1000, 0) << QPointF(-1000, 0); + QTest::newRow("7") << true << QPoint(-100, 0) << QPointF(-100, 0); + QTest::newRow("8") << true << QPoint(-10, 0) << QPointF(-10, 0); + QTest::newRow("0-") << false << QPoint() << QPointF(); + QTest::newRow("1-") << false << QPoint(10, 0) << QPointF(10, 0); + QTest::newRow("2-") << false << QPoint(100, 0) << QPointF(100, 0); + QTest::newRow("3-") << false << QPoint(1000, 0) << QPointF(1000, 0); + QTest::newRow("4-") << false << QPoint(10000, 0) << QPointF(10000, 0); + QTest::newRow("5-") << false << QPoint(-10000, 0) << QPointF(-10000, 0); + QTest::newRow("6-") << false << QPoint(-1000, 0) << QPointF(-1000, 0); + QTest::newRow("7-") << false << QPoint(-100, 0) << QPointF(-100, 0); + QTest::newRow("8-") << false << QPoint(-10, 0) << QPointF(-10, 0); } void tst_QGraphicsProxyWidget::childPos() @@ -2681,7 +2678,6 @@ void tst_QGraphicsProxyWidget::childPos() QFETCH(bool, moveCombo); QFETCH(QPoint, comboPos); QFETCH(QPointF, proxyPos); - QFETCH(QPointF, menuPos); QGraphicsScene scene; diff --git a/tests/auto/widgets/graphicsview/qgraphicsview/tst_qgraphicsview.cpp b/tests/auto/widgets/graphicsview/qgraphicsview/tst_qgraphicsview.cpp index ad204f2..a5dc0c6 100644 --- a/tests/auto/widgets/graphicsview/qgraphicsview/tst_qgraphicsview.cpp +++ b/tests/auto/widgets/graphicsview/qgraphicsview/tst_qgraphicsview.cpp @@ -1369,7 +1369,6 @@ void tst_QGraphicsView::itemsInRect() view.setSceneRect(-10000, -10000, 20000, 20000); view.show(); - QPoint centerPoint = view.viewport()->rect().center(); QRect leftRect = view.mapFromScene(-30, -10, 20, 20).boundingRect(); QRect rightRect = view.mapFromScene(30, -10, 20, 20).boundingRect(); @@ -1490,7 +1489,6 @@ void tst_QGraphicsView::itemsInPoly() view.setSceneRect(-10000, -10000, 20000, 20000); view.show(); - QPoint centerPoint = view.viewport()->rect().center(); QPolygon leftPoly = view.mapFromScene(QRectF(-30, -10, 20, 20)); QPolygon rightPoly = view.mapFromScene(QRectF(30, -10, 20, 20)); @@ -1534,7 +1532,6 @@ void tst_QGraphicsView::itemsInPath() view.setSceneRect(-10000, -10000, 20000, 20000); view.show(); - QPoint centerPoint = view.viewport()->rect().center(); QPainterPath leftPath; leftPath.addEllipse(QRect(view.mapFromScene(-30, -10), QSize(20, 20))); @@ -1794,7 +1791,6 @@ void tst_QGraphicsView::mapToScenePath() view.translate(10, 10); view.setFixedSize(300, 300); view.show(); - QPoint center = view.viewport()->rect().center(); QRect rect(QPoint(10, 0), QSize(10, 10)); QPainterPath path; diff --git a/tests/auto/widgets/itemviews/qstringlistmodel/tst_qstringlistmodel.cpp b/tests/auto/widgets/itemviews/qstringlistmodel/tst_qstringlistmodel.cpp index 2542496..32c26b5 100644 --- a/tests/auto/widgets/itemviews/qstringlistmodel/tst_qstringlistmodel.cpp +++ b/tests/auto/widgets/itemviews/qstringlistmodel/tst_qstringlistmodel.cpp @@ -169,7 +169,7 @@ void tst_QStringListModel::rowsAboutToBeRemoved_rowsRemoved_data() QStringList strings3; strings3 << "One" << "Two" << "Three" << "Four" << "Five"; QStringList aboutto3; aboutto3 << "One" << "Two" << "Three" << "Four" << "Five"; - QStringList res3; res3 ; + QStringList res3; QTest::newRow( "data3" ) << strings3 << 0 << 5 << aboutto3 << res3; /* Not sure if this is a valid test */ diff --git a/tests/auto/widgets/itemviews/qtreewidget/tst_qtreewidget.cpp b/tests/auto/widgets/itemviews/qtreewidget/tst_qtreewidget.cpp index 71a82cf..ca358d3 100644 --- a/tests/auto/widgets/itemviews/qtreewidget/tst_qtreewidget.cpp +++ b/tests/auto/widgets/itemviews/qtreewidget/tst_qtreewidget.cpp @@ -3280,8 +3280,6 @@ void tst_QTreeWidget::taskQTBUG2844_visualItemRect() tree.setColumnCount(3); QTreeWidgetItem item(&tree); - QRect itemRect = tree.visualItemRect(&item); - QRect rectCol0 = tree.visualRect(tree.indexFromItem(&item, 0)); QRect rectCol1 = tree.visualRect(tree.indexFromItem(&item, 1)); QRect rectCol2 = tree.visualRect(tree.indexFromItem(&item, 2)); diff --git a/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp b/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp index 67df4df..fe02d1a 100644 --- a/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp +++ b/tests/auto/widgets/kernel/qwidget/tst_qwidget.cpp @@ -5015,8 +5015,6 @@ void tst_QWidget::moveChild() parent.setGeometry(60, 60, 150, 150); #endif child.setGeometry(25, 25, 50, 50); - QPoint childOffset = child.mapToGlobal(QPoint()); - parent.show(); QTest::qWaitForWindowShown(&parent); QTest::qWait(30); @@ -6970,9 +6968,6 @@ void tst_QWidget::render_worldTransform() painter.translate(105, 5); painter.rotate(90); - const QTransform worldTransform = painter.worldTransform(); - const QTransform deviceTransform = painter.deviceTransform(); - // Render widgets onto image. widget.render(&painter); #ifdef RENDER_DEBUG @@ -7100,7 +7095,6 @@ void tst_QWidget::repaintWhenChildDeleted() w.r = QRegion(); { - const QPoint tlwOffset = w.geometry().topLeft(); ColorWidget child(&w, Qt::blue); child.setGeometry(10, 10, 10, 10); child.show(); diff --git a/tests/auto/widgets/widgets/qcalendarwidget/tst_qcalendarwidget.cpp b/tests/auto/widgets/widgets/qcalendarwidget/tst_qcalendarwidget.cpp index 50dc6a1..e4c1df2 100644 --- a/tests/auto/widgets/widgets/qcalendarwidget/tst_qcalendarwidget.cpp +++ b/tests/auto/widgets/widgets/qcalendarwidget/tst_qcalendarwidget.cpp @@ -180,7 +180,6 @@ void tst_QCalendarWidget::buttonClickCheck() object.setGeometry(0,0,size.width(), size.height()); object.show(); - QRect rect = object.geometry(); QDate selectedDate(2005, 1, 1); //click on the month buttons int month = object.monthShown(); diff --git a/tests/auto/widgets/widgets/qmainwindow/tst_qmainwindow.cpp b/tests/auto/widgets/widgets/qmainwindow/tst_qmainwindow.cpp index 952078c..1fbd0f0 100644 --- a/tests/auto/widgets/widgets/qmainwindow/tst_qmainwindow.cpp +++ b/tests/auto/widgets/widgets/qmainwindow/tst_qmainwindow.cpp @@ -294,7 +294,6 @@ void tst_QMainWindow::iconSize() // the default is determined by the style const int metric = mw.style()->pixelMetric(QStyle::PM_ToolBarIconSize); - const QSize defaultIconSize = QSize(metric, metric); const QSize smallIconSize = QSize(metric / 2, metric / 2); const QSize largeIconSize = QSize(metric * 2, metric * 2); diff --git a/tests/auto/widgets/widgets/qtextedit/tst_qtextedit.cpp b/tests/auto/widgets/widgets/qtextedit/tst_qtextedit.cpp index ab128a2..0a76c9a 100644 --- a/tests/auto/widgets/widgets/qtextedit/tst_qtextedit.cpp +++ b/tests/auto/widgets/widgets/qtextedit/tst_qtextedit.cpp @@ -2026,9 +2026,6 @@ void tst_QTextEdit::fullWidthSelection2() widget.setExtraSelections(selections); compareWidgetAndImage(widget, "fullWidthSelection/nowrap_long.png"); - - QTextLine line = widget.document()->begin().layout()->lineAt(0); - // qDebug() << line.width(); } void tst_QTextEdit::compareWidgetAndImage(QTextEdit &widget, const QString &imageFileName)