Cleanup code to comply with Qt style

- Correct placement of curly braces
- Remove commented-out code
- Remove superfluous qDebug statements
- Add missing QT_BEGIN_NAMESPACE declarations
- Add move semantics
- Add cleanup handling to QWebSocketServerPrivate
- Add error handling to handshake response, QWebSocketServerPrivate,
  QWebSocketServer, dataprocessor

Change-Id: I0690dc2c444fd6fc0db974d1459bf41bd8c31d40
Reviewed-by: Kurt Pattyn <pattyn.kurt@gmail.com>

1 files changed, 18 insertions, 3 deletions

diff --git a/src/websockets/qwebsocketserver.cpp b/src/websockets/qwebsocketserver.cpp
index 461e75f..3198c13 100644
--- a/src/websockets/qwebsocketserver.cpp
+++ b/src/websockets/qwebsocketserver.cpp
@@ -103,6 +103,14 @@
 */
 
 /*!
+    \fn void QWebSocketServer::serverError(QNetworkProtocol::CloseCode closeCode)
+    This signal is emitted when an error occurs during the setup of a web socket connection.
+    The \a closeCode parameter describes the type of error that occurred
+
+    \sa errorString()
+*/
+
+/*!
     \fn void QWebSocketServer::newConnection()
     This signal is emitted every time a new connection is available.
 
@@ -110,6 +118,13 @@
 */
 
 /*!
+    \fn void QWebSocketServer::closed()
+    This signal is emitted when the server closed it's connection.
+
+    \sa close()
+*/
+
+/*!
     \fn void QWebSocketServer::originAuthenticationRequired(QWebSocketCorsAuthenticator *authenticator)
     This signal is emitted when a new connection is requested.
     The slot connected to this signal should indicate whether the origin (which can be determined by the origin() call)
@@ -381,7 +396,7 @@ QWebSocketServer::SecureMode QWebSocketServer::secureMode() const
     Returns an error code for the last error that occurred.
     \sa errorString()
  */
-QAbstractSocket::SocketError QWebSocketServer::serverError() const
+QWebSocketProtocol::CloseCode QWebSocketServer::error() const
 {
     Q_D(const QWebSocketServer);
     return d->serverError();
@@ -468,7 +483,7 @@ QList<QWebSocketProtocol::Version> QWebSocketServer::supportedVersions() const
 /*!
   Returns a list of websocket subprotocols that this server supports.
  */
-QList<QString> QWebSocketServer::supportedProtocols() const
+QStringList QWebSocketServer::supportedProtocols() const
 {
     Q_D(const QWebSocketServer);
     return d->supportedProtocols();
@@ -477,7 +492,7 @@ QList<QString> QWebSocketServer::supportedProtocols() const
 /*!
   Returns a list of websocket extensions that this server supports.
  */
-QList<QString> QWebSocketServer::supportedExtensions() const
+QStringList QWebSocketServer::supportedExtensions() const
 {
     Q_D(const QWebSocketServer);
     return d->supportedExtensions();