From ceca141d0cfb210859db28c8f7725790fe23a767 Mon Sep 17 00:00:00 2001 From: Thiago Macieira Date: Thu, 20 Sep 2012 15:46:41 +0200 Subject: [PATCH] Remove the annoying qWarning when QUrl::setPort is out of range That's what we have QUrl::errorString() for. This will become evident especially now that QUrl::toString() / toEncoded() return empty if there are errors. Change-Id: I64a84e9c6ee57c0fc38cc0c58f5286ddc1248d1f Reviewed-by: Shane Kearns Reviewed-by: David Faure --- src/corelib/io/qurl.cpp | 1 - tests/auto/corelib/io/qurl/tst_qurl.cpp | 3 --- 2 files changed, 4 deletions(-) diff --git a/src/corelib/io/qurl.cpp b/src/corelib/io/qurl.cpp index 3c82a2659f..e78d810bdb 100644 --- a/src/corelib/io/qurl.cpp +++ b/src/corelib/io/qurl.cpp @@ -2105,7 +2105,6 @@ void QUrl::setPort(int port) detach(); if (port < -1 || port > 65535) { - qWarning("QUrl::setPort: Out of range"); port = -1; d->sectionHasError |= QUrlPrivate::Port; d->errorCode = QUrlPrivate::InvalidPortError; diff --git a/tests/auto/corelib/io/qurl/tst_qurl.cpp b/tests/auto/corelib/io/qurl/tst_qurl.cpp index c1fb6f702a..03bc64e753 100644 --- a/tests/auto/corelib/io/qurl/tst_qurl.cpp +++ b/tests/auto/corelib/io/qurl/tst_qurl.cpp @@ -183,11 +183,9 @@ void tst_QUrl::getSetCheck() obj1.setPort(0); QCOMPARE(0, obj1.port()); - QTest::ignoreMessage(QtWarningMsg, "QUrl::setPort: Out of range"); obj1.setPort(INT_MIN); QCOMPARE(-1, obj1.port()); // Out of range, -1 - QTest::ignoreMessage(QtWarningMsg, "QUrl::setPort: Out of range"); obj1.setPort(INT_MAX); QCOMPARE(-1, obj1.port()); // Out of range, -1 @@ -2443,7 +2441,6 @@ void tst_QUrl::setPort() QCOMPARE(url.port(), -1); QCOMPARE(url.toString(), QString()); url.setPort(80); - QTest::ignoreMessage(QtWarningMsg, "QUrl::setPort: Out of range"); url.setPort(65536); QCOMPARE(url.port(), -1); QVERIFY(url.errorString().contains("out of range"));