Update code style, fix compiler warnings

- Use nullptr instead of NULL for better warnings in case of mistakes
- Get rid of VLAs which are not in C++11 actually
- Fix implicit signed <-> unsigned mismatches by adding checks and casts

1 files changed, 5 insertions, 5 deletions

diff --git a/src/server/net/sslserver.cpp b/src/server/net/sslserver.cpp
index 0e0639e..c11f0ae 100644
--- a/src/server/net/sslserver.cpp
+++ b/src/server/net/sslserver.cpp
@@ -20,7 +20,7 @@
 #include "certmanager.h"
 #include <unistd.h>
 
-SslServer::SslServer() : QTcpServer(NULL)
+SslServer::SslServer() : QTcpServer(nullptr)
 {
 	_tmr = startTimer(5123);
 }
@@ -43,10 +43,10 @@ void SslServer::incomingConnection(qintptr socketDescriptor)
 		if (++certFails > 5) {
 			CertManager::fatal();
 		}
-		::close(socketDescriptor);
+		::close((int)socketDescriptor);
 		return;
 	}
-	QSslSocket *serverSocket = new QSslSocket(NULL);
+	QSslSocket *serverSocket = new QSslSocket(nullptr);
 	connect(serverSocket, SIGNAL(sslErrors(const QList<QSslError> &)), this, SLOT(sslErrors(const QList<QSslError> &)));
 	connect(serverSocket, SIGNAL(disconnected()), this, SLOT(sock_closed()));
 	connect(serverSocket, SIGNAL(error(QAbstractSocket::SocketError)), this, SLOT(sock_error(QAbstractSocket::SocketError)));
@@ -70,7 +70,7 @@ void SslServer::incomingConnection(qintptr socketDescriptor)
 	}
 }
 
-void SslServer::sslErrors(const QList<QSslError> &errors)
+void SslServer::sslErrors(const QList<QSslError>& /* errors */)
 {
 	/*
 	qDebug() << "Client caused sslErrors before connection:";
@@ -134,6 +134,6 @@ QTcpSocket* SslServer::nextPendingConnection()
 			return sock;
 		}
 	}
-	return NULL;
+	return nullptr;
 }