From 895816988e0d03e30f4d54bf235af8d201aaad67 Mon Sep 17 00:00:00 2001 From: Manuel Nickschas Date: Tue, 4 Sep 2018 23:47:44 +0200 Subject: [PATCH] clang-tidy: Don't make methods virtual for no reason in SslServer SslServer was calling virtual methods during construction, which is dangerous. Turns out those methods don't even need to be virtual, so unvirtualize them. --- src/core/sslserver.h | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/src/core/sslserver.h b/src/core/sslserver.h index 16ab183b..609ec77a 100644 --- a/src/core/sslserver.h +++ b/src/core/sslserver.h @@ -18,8 +18,7 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. * ***************************************************************************/ -#ifndef SSLSERVER_H -#define SSLSERVER_H +#pragma once #ifdef HAVE_SSL @@ -36,12 +35,12 @@ class SslServer : public QTcpServer public: SslServer(QObject *parent = 0); - virtual inline bool hasPendingConnections() const { return !_pendingConnections.isEmpty(); } - virtual QTcpSocket *nextPendingConnection(); + bool hasPendingConnections() const override { return !_pendingConnections.isEmpty(); } + QTcpSocket *nextPendingConnection() override; - virtual inline const QSslCertificate &certificate() const { return _cert; } - virtual inline const QSslKey &key() const { return _key; } - virtual inline bool isCertValid() const { return _isCertValid; } + const QSslCertificate &certificate() const { return _cert; } + const QSslKey &key() const { return _key; } + bool isCertValid() const { return _isCertValid; } /** * Reloads SSL certificates used for connections @@ -55,12 +54,12 @@ public: protected: #if QT_VERSION >= 0x050000 - virtual void incomingConnection(qintptr socketDescriptor); + void incomingConnection(qintptr socketDescriptor) override; #else - virtual void incomingConnection(int socketDescriptor); + void incomingConnection(int socketDescriptor) override; #endif - virtual bool setCertificate(const QString &path, const QString &keyPath); + bool setCertificate(const QString &path, const QString &keyPath); private: /** @@ -87,5 +86,3 @@ private: #endif //HAVE_SSL - -#endif //SSLSERVER_H -- 2.20.1