Details
-
Task
-
Resolution: Done
-
P3: Somewhat important
-
6.4
-
None
-
All
-
3
-
55cef3a13c (qt/qtwebsockets/dev) 55cef3a13c (qt/tqtc-qtwebsockets/dev)
-
Foundation Sprint 61, Foundation Sprint 62
Description
To match other classes in Qt (e.g QIODevice, QNetworkReply), the overloaded error() signal should go away and be replaced by errorOccurred(). Removal can only be done in the next major version of Qt, so until then it can only be marked as deprecated.
Since error() is overloaded users need to use qOverload when connect()ing to the signal, which is quite ugly and inconvenient compared to when it is not needed.
Attachments
| For Gerrit Dashboard: QTBUG-101756 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 422542,8 | Replace QWebSocket::error signal with QWebSocket::errorOccurred | dev | qt/qtwebsockets | Status: MERGED | +2 | 0 |