From 0011a4510265a13b16f04bce8f16cc9381c22e88 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?M=C3=A5rten=20Nordheim?= Date: Wed, 19 Aug 2020 16:42:58 +0200 Subject: QAbstractSocket: Resolve remaining Qt6 TODOs Change-Id: Id257f0721c1cd5fcbafa9297bae0251a2d68e366 Reviewed-by: Timur Pocheptsov --- examples/network/torrent/peerwireclient.cpp | 7 ++--- examples/network/torrent/peerwireclient.h | 5 ++-- .../code/src_network_socket_qabstractsocket.cpp | 14 +-------- src/network/socket/qabstractsocket.cpp | 35 +--------------------- src/network/socket/qabstractsocket.h | 10 ++----- 5 files changed, 11 insertions(+), 60 deletions(-) diff --git a/examples/network/torrent/peerwireclient.cpp b/examples/network/torrent/peerwireclient.cpp index c30abd0e13..177568d402 100644 --- a/examples/network/torrent/peerwireclient.cpp +++ b/examples/network/torrent/peerwireclient.cpp @@ -397,12 +397,11 @@ bool PeerWireClient::canTransferMore() const || !outgoingBuffer.isEmpty() || !pendingBlocks.isEmpty(); } -void PeerWireClient::connectToHost(const QHostAddress &address, - quint16 port, OpenMode openMode) - +void PeerWireClient::connectToHost(const QString &address, quint16 port, OpenMode openMode, + NetworkLayerProtocol protocol) { setOpenMode(openMode); - socket.connectToHost(address, port, openMode); + socket.connectToHost(address, port, openMode, protocol); } void PeerWireClient::diconnectFromHost() diff --git a/examples/network/torrent/peerwireclient.h b/examples/network/torrent/peerwireclient.h index 75edf8ee77..2d64329854 100644 --- a/examples/network/torrent/peerwireclient.h +++ b/examples/network/torrent/peerwireclient.h @@ -128,8 +128,9 @@ public: void setReadBufferSize(qint64 size) override; - void connectToHost(const QHostAddress &address, - quint16 port, OpenMode openMode = ReadWrite) override; + using QTcpSocket::connectToHost; + void connectToHost(const QString &address, quint16 port, OpenMode openMode = ReadWrite, + NetworkLayerProtocol protocol = AnyIPProtocol) override; void diconnectFromHost(); signals: diff --git a/src/network/doc/snippets/code/src_network_socket_qabstractsocket.cpp b/src/network/doc/snippets/code/src_network_socket_qabstractsocket.cpp index e19cb40666..e03d8ca7be 100644 --- a/src/network/doc/snippets/code/src_network_socket_qabstractsocket.cpp +++ b/src/network/doc/snippets/code/src_network_socket_qabstractsocket.cpp @@ -65,17 +65,5 @@ if (socket->state() == QAbstractSocket::UnconnectedState //! [2] - // This slot is connected to QAbstractSocket::readyRead() - void SocketClass::readyReadSlot() - { - while (!socket.atEnd()) { - QByteArray data = socket.read(100); - .... - } - } -//! [2] - - -//! [3] socket->setProxy(QNetworkProxy::NoProxy); -//! [3] +//! [2] diff --git a/src/network/socket/qabstractsocket.cpp b/src/network/socket/qabstractsocket.cpp index 3bde2215ad..f3885a22fe 100644 --- a/src/network/socket/qabstractsocket.cpp +++ b/src/network/socket/qabstractsocket.cpp @@ -1864,22 +1864,6 @@ QString QAbstractSocket::peerName() const return d->peerName.isEmpty() ? d->hostName : d->peerName; } -/*! - Returns \c true if a line of data can be read from the socket; - otherwise returns \c false. - - \sa readLine() -*/ -bool QAbstractSocket::canReadLine() const -{ - bool hasLine = QIODevice::canReadLine(); -#if defined (QABSTRACTSOCKET_DEBUG) - qDebug("QAbstractSocket::canReadLine() == %s, buffer size = %lld, size = %lld", - hasLine ? "true" : "false", d_func()->buffer.size(), d_func()->buffer.size()); -#endif - return hasLine; -} - /*! Returns the native socket descriptor of the QAbstractSocket object if this is available; otherwise returns -1. @@ -2413,23 +2397,6 @@ bool QAbstractSocket::isSequential() const return true; } -/*! \reimp - - Returns \c true if no more data is currently - available for reading; otherwise returns \c false. - - This function is most commonly used when reading data from the - socket in a loop. For example: - - \snippet code/src_network_socket_qabstractsocket.cpp 2 - - \sa bytesAvailable(), readyRead() - */ -bool QAbstractSocket::atEnd() const -{ - return QIODevice::atEnd(); -} - /*! This function writes as much as possible from the internal write buffer to the underlying network socket, without blocking. If any data was written, @@ -2882,7 +2849,7 @@ void QAbstractSocket::setSocketError(SocketError socketError) To disable the use of a proxy for this socket, use the QNetworkProxy::NoProxy proxy type: - \snippet code/src_network_socket_qabstractsocket.cpp 3 + \snippet code/src_network_socket_qabstractsocket.cpp 2 The default value for the proxy is QNetworkProxy::DefaultProxy, which means the socket will use the application settings: if a diff --git a/src/network/socket/qabstractsocket.h b/src/network/socket/qabstractsocket.h index 865f667d46..bf1ff7e9a2 100644 --- a/src/network/socket/qabstractsocket.h +++ b/src/network/socket/qabstractsocket.h @@ -146,13 +146,12 @@ public: PauseModes pauseMode() const; void setPauseMode(PauseModes pauseMode); - // ### Qt6: make the first one virtual - bool bind(const QHostAddress &address, quint16 port = 0, BindMode mode = DefaultForPlatform); + virtual bool bind(const QHostAddress &address, quint16 port = 0, + BindMode mode = DefaultForPlatform); bool bind(quint16 port = 0, BindMode mode = DefaultForPlatform); - // ### Qt6: de-virtualize connectToHost(QHostAddress) overload virtual void connectToHost(const QString &hostName, quint16 port, OpenMode mode = ReadWrite, NetworkLayerProtocol protocol = AnyIPProtocol); - virtual void connectToHost(const QHostAddress &address, quint16 port, OpenMode mode = ReadWrite); + void connectToHost(const QHostAddress &address, quint16 port, OpenMode mode = ReadWrite); virtual void disconnectFromHost(); bool isValid() const; @@ -160,8 +159,6 @@ public: qint64 bytesAvailable() const override; qint64 bytesToWrite() const override; - bool canReadLine() const override; // ### Qt6: remove me - quint16 localPort() const; QHostAddress localAddress() const; quint16 peerPort() const; @@ -187,7 +184,6 @@ public: // from QIODevice void close() override; bool isSequential() const override; - bool atEnd() const override; // ### Qt6: remove me bool flush(); // for synchronous access -- cgit v1.2.3