From 92eac408783fd4b1e2db2759c3212b580ff24205 Mon Sep 17 00:00:00 2001 From: Aaron McCarthy Date: Wed, 3 Feb 2010 15:48:32 +1000 Subject: Fix compile warnings. --- src/network/access/qnetworkaccessmanager.cpp | 63 +++++++++++----------------- 1 file changed, 25 insertions(+), 38 deletions(-) diff --git a/src/network/access/qnetworkaccessmanager.cpp b/src/network/access/qnetworkaccessmanager.cpp index dbcdddf..4e28641 100644 --- a/src/network/access/qnetworkaccessmanager.cpp +++ b/src/network/access/qnetworkaccessmanager.cpp @@ -805,28 +805,24 @@ void QNetworkAccessManagerPrivate::_q_replyFinished() emit q->finished(reply); if (deferredMigration) { - bool repliesPending = false; foreach (QObject *child, q->children()) { - if (child != reply && child->inherits("QNetworkReplyImpl")) { - QNetworkReplyImpl *replyImpl = qobject_cast(child); - qDebug() << "reply state is" << replyImpl->d_func()->state; - switch (replyImpl->d_func()->state) { - case QNetworkReplyImplPrivate::Idle: - case QNetworkReplyImplPrivate::Finished: - case QNetworkReplyImplPrivate::Aborted: - break; - case QNetworkReplyImplPrivate::Buffering: - case QNetworkReplyImplPrivate::Working: - repliesPending = true; - break; - } + if (child == reply) + continue; + + QNetworkReplyImpl *replyImpl = qobject_cast(child); + if (!replyImpl) + continue; + + QNetworkReplyImplPrivate::State state = replyImpl->d_func()->state; + if (state == QNetworkReplyImplPrivate::Buffering || + state == QNetworkReplyImplPrivate::Working) { + return; } } - if (!repliesPending) { - deferredMigration = false; - qDebug() << "Migrating as there are no pending replies."; - session->migrate(); - } + + deferredMigration = false; + qDebug() << "Migrating as there are no pending replies."; + session->migrate(); } } @@ -1108,8 +1104,6 @@ void QNetworkAccessManagerPrivate::_q_sessionError(QNetworkSession::SessionError void QNetworkAccessManagerPrivate::_q_sessionStateChanged(QNetworkSession::State state) { - Q_Q(QNetworkAccessManager); - qDebug() << "session state changed to" << state; } @@ -1144,32 +1138,25 @@ void QNetworkAccessManagerPrivate::_q_sessionNewConfigurationActivated() } } -void QNetworkAccessManagerPrivate::_q_sessionPreferredConfigurationChanged(const QNetworkConfiguration &config, bool isSeamless) +void QNetworkAccessManagerPrivate::_q_sessionPreferredConfigurationChanged(const QNetworkConfiguration &, bool) { Q_Q(QNetworkAccessManager); - deferredMigration = false; foreach (QObject *child, q->children()) { - if (child->inherits("QNetworkReplyImpl")) { - QNetworkReplyImpl *replyImpl = qobject_cast(child); - qDebug() << "reply state is" << replyImpl->d_func()->state; - switch (replyImpl->d_func()->state) { - case QNetworkReplyImplPrivate::Idle: - case QNetworkReplyImplPrivate::Finished: - case QNetworkReplyImplPrivate::Aborted: - break; - case QNetworkReplyImplPrivate::Buffering: - case QNetworkReplyImplPrivate::Working: + QNetworkReplyImpl *replyImpl = qobject_cast(child); + if (replyImpl) { + QNetworkReplyImplPrivate::State state = replyImpl->d_func()->state; + if (state == QNetworkReplyImplPrivate::Buffering || + state == QNetworkReplyImplPrivate::Working) { deferredMigration = true; - break; + return; } } } - if (!deferredMigration) { - qDebug() << "Migrating as there are no pending replies."; - session->migrate(); - } + deferredMigration = false; + qDebug() << "Migrating as there are no pending replies."; + session->migrate(); } QT_END_NAMESPACE -- cgit v0.12