From bb234e238c777b1a708912765d6899bf9061bcb8 Mon Sep 17 00:00:00 2001 From: Shane Kearns Date: Mon, 30 Apr 2012 19:14:06 +0100 Subject: Rename QAbstractSocket::PauseOnNotify -> PauseOnSslErrors Although we created an enum for pause modes to make 5.x binary compatible with 5.0, the enum value is not well named. In 5.1, we propose to add PauseOnProxyAuthentication to the enum. PauseOnNotify is not clear what it means, while PauseOnSslErrors is. Any new notification in a minor release would need a new enum value otherwise applications would get pauses they did not expect. Task-number: QTBUG-19032 Change-Id: I4dbb7467663b37ca7f0551d24a31bc013968bedc Reviewed-by: Richard J. Moore Reviewed-by: Lars Knoll --- src/network/socket/qabstractsocket.cpp | 8 ++++---- src/network/socket/qabstractsocket.h | 2 +- src/network/ssl/qsslsocket.cpp | 2 +- src/network/ssl/qsslsocket_openssl.cpp | 2 +- tests/auto/network/ssl/qsslsocket/tst_qsslsocket.cpp | 2 +- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/network/socket/qabstractsocket.cpp b/src/network/socket/qabstractsocket.cpp index 0c4eed8de9..a6eb3626a0 100644 --- a/src/network/socket/qabstractsocket.cpp +++ b/src/network/socket/qabstractsocket.cpp @@ -434,12 +434,12 @@ This enum describes the behavior of when the socket should hold back with continuing data transfer. + The only notification currently supported is QSslSocket::sslErrors(). \value PauseNever Do not pause data transfer on the socket. This is the default and matches the behaviour of Qt 4. - \value PauseOnNotify Pause data transfer on the socket upon receiving a - notification. The only notification currently supported is - QSslSocket::sslErrors(). + \value PauseOnSslErrors Pause data transfer on the socket upon receiving an + SSL error notification. I.E. QSslSocket::sslErrors(). */ #include "qabstractsocket.h" @@ -1404,7 +1404,7 @@ QAbstractSocket::PauseModes QAbstractSocket::pauseMode() const \since 5.0 Controls whether to pause upon receiving a notification. The only notification - currently supported is QSslSocket::sslErrors(). If set to PauseOnNotify, + currently supported is QSslSocket::sslErrors(). If set to PauseOnSslErrors, data transfer on the socket will be paused and needs to be enabled explicitly again by calling resume(). By default this option is set to PauseNever. diff --git a/src/network/socket/qabstractsocket.h b/src/network/socket/qabstractsocket.h index 7a9f70b47e..dedde9649a 100644 --- a/src/network/socket/qabstractsocket.h +++ b/src/network/socket/qabstractsocket.h @@ -127,7 +127,7 @@ public: Q_DECLARE_FLAGS(BindMode, BindFlag) enum PauseMode { PauseNever = 0x0, - PauseOnNotify = 0x1 + PauseOnSslErrors = 0x1 }; Q_DECLARE_FLAGS(PauseModes, PauseMode) diff --git a/src/network/ssl/qsslsocket.cpp b/src/network/ssl/qsslsocket.cpp index cfc59fd575..3445f13f39 100644 --- a/src/network/ssl/qsslsocket.cpp +++ b/src/network/ssl/qsslsocket.cpp @@ -361,7 +361,7 @@ QSslSocket::~QSslSocket() \since 5.0 Continues data transfer on the socket after it has been paused. If - "setPauseMode(QAbstractSocket::PauseOnNotify);" has been called on + "setPauseMode(QAbstractSocket::PauseOnSslErrors);" has been called on this socket and a sslErrors() signal is received, calling this method is necessary for the socket to continue. diff --git a/src/network/ssl/qsslsocket_openssl.cpp b/src/network/ssl/qsslsocket_openssl.cpp index eddedac486..d193449699 100644 --- a/src/network/ssl/qsslsocket_openssl.cpp +++ b/src/network/ssl/qsslsocket_openssl.cpp @@ -1299,7 +1299,7 @@ bool QSslSocketBackendPrivate::checkSslErrors() bool doEmitSslError = !verifyErrorsHaveBeenIgnored(); // check whether we need to emit an SSL handshake error if (doVerifyPeer && doEmitSslError) { - if (q->pauseMode() & QAbstractSocket::PauseOnNotify) { + if (q->pauseMode() & QAbstractSocket::PauseOnSslErrors) { pauseSocketNotifiers(q); paused = true; } else { diff --git a/tests/auto/network/ssl/qsslsocket/tst_qsslsocket.cpp b/tests/auto/network/ssl/qsslsocket/tst_qsslsocket.cpp index 34bc361e4b..e134f346e2 100644 --- a/tests/auto/network/ssl/qsslsocket/tst_qsslsocket.cpp +++ b/tests/auto/network/ssl/qsslsocket/tst_qsslsocket.cpp @@ -2158,7 +2158,7 @@ void tst_QSslSocket::resume() QFETCH(bool, expectSuccess); QSslSocket socket; - socket.setPauseMode(QAbstractSocket::PauseOnNotify); + socket.setPauseMode(QAbstractSocket::PauseOnSslErrors); QSignalSpy sslErrorSpy(&socket, SIGNAL(sslErrors(QList))); QSignalSpy encryptedSpy(&socket, SIGNAL(encrypted())); -- cgit v1.2.3