Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-101756

The QWebSocket::error signal should be renamed errorOccurred

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Done
    • Icon: P3: Somewhat important P3: Somewhat important
    • 6.5.0 Beta1
    • 6.4
    • WebSockets
    • None
    • All
    • 3
    • 55cef3a13c (qt/qtwebsockets/dev) 55cef3a13c (qt/tqtc-qtwebsockets/dev)
    • Foundation Sprint 61, Foundation Sprint 62

      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.

        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

            oyheskes Øystein Heskestad
            manordheim Mårten Nordheim
            Vladimir Minenko Vladimir Minenko
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes