Fix some compiler warnings in tests.

- Unused variables
- Deprecated conversion from const char * to char *.

Change-Id: Iea0b9c4613ea74cead6d95ba12ad1028f531cbff
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>
This commit is contained in:
Friedemann Kleint 2012-03-23 11:44:19 +01:00 committed by Qt by Nokia
parent 0ee1b4a1de
commit 2c4845ce33
8 changed files with 20 additions and 14 deletions

View File

@ -718,6 +718,11 @@ void tst_QString::acc_01()
}
}
#ifdef Q_CC_GNU
# pragma GCC diagnostic push
# pragma GCC diagnostic ignored "-Wformat-security"
#endif
void tst_QString::isNull()
{
QString a;
@ -728,6 +733,10 @@ void tst_QString::isNull()
QVERIFY(!a.isNull());
}
#ifdef Q_CC_GNU
# pragma GCC diagnostic pop
#endif
void tst_QString::isEmpty()
{
QString a;

View File

@ -72,7 +72,7 @@ public:
if (n == 0)
break;
qDebug() << "Read" << str;
if ("exit" == str)
if (!qstrcmp(str, "exit"))
qApp->quit();
if (socket->write(str, 100) < 0) {

View File

@ -76,7 +76,7 @@ private slots:
int main()
{
char *argv[] = {"appName", "slotName"};
char *argv[] = { const_cast<char *>("appName"), const_cast<char *>("slotName") };
int argc = 2;
tst_TestA testA;

View File

@ -65,7 +65,7 @@ void tst_StrCmp::compareCharStars() const
QCOMPARE(str1, "foo");
QCOMPARE(str1, str1);
char *str2 = "foo";
char *str2 = const_cast<char *>("foo");
QCOMPARE("foo", str2);
QCOMPARE(str2, "foo");
QCOMPARE(str2, str2);
@ -88,8 +88,8 @@ void tst_StrCmp::compareByteArray() const
QCOMPARE(ba.constData(), "bar");
QCOMPARE(ba.constData(), "foo");
char *bar = "bar";
char *foo = "foo";
char *bar = const_cast<char *>("bar");
char *foo = const_cast<char *>("foo");
QEXPECT_FAIL("", "Next test should fail", Continue);
QCOMPARE(ba.data(), bar);

View File

@ -128,7 +128,7 @@ TestCompiler::~TestCompiler()
bool TestCompiler::errorOut()
{
qDebug(qPrintable(testOutput_.join("\n")));
qDebug("%s", qPrintable(testOutput_.join(QStringLiteral("\n"))));
return false;
}

View File

@ -287,17 +287,15 @@ void tst_QGraphicsLayoutItem::setMaximumSize()
void tst_QGraphicsLayoutItem::setMinimumSize_data()
{
QTest::addColumn<QSizeF>("size");
QTest::addColumn<QSizeF>("outputSize");
QTest::newRow("-1") << QSizeF(-1, -1) << QSizeF(0, 0);
QTest::newRow("0") << QSizeF(0, 0) << QSizeF(0, 0);
QTest::newRow("10") << QSizeF(10, 10) << QSizeF(10, 10);
QTest::newRow("-1") << QSizeF(-1, -1);
QTest::newRow("0") << QSizeF(0, 0);
QTest::newRow("10") << QSizeF(10, 10);
}
// void setMinimumSize(QSizeF const& size) public
void tst_QGraphicsLayoutItem::setMinimumSize()
{
QFETCH(QSizeF, size);
QFETCH(QSizeF, outputSize);
SubQGraphicsLayoutItem layoutItem;
QSizeF oldSize = layoutItem.minimumSize();

View File

@ -238,10 +238,9 @@ public:
openedEditor = new QWidget(parent);
return openedEditor;
}
void destroyEditor(QWidget *editor, const QModelIndex &index) const
void destroyEditor(QWidget *editor, const QModelIndex &) const
{
calledVirtualDtor = true;
// QAbstractItemDelegate::destroyEditor(editor,index);
editor->deleteLater();
}

View File

@ -118,9 +118,9 @@ class ServerAndClient : public QObject
public:
ServerAndClient(QEventLoop &ev) : success(false)
, eventLoop(ev)
, isBody(false)
, bodyBytesRead(0)
, bodyLength(-1)
, isBody(false)
{
setObjectName("serverAndClient");
tcpServer = new QTcpServer(this);