Fix some clang-tidy warnings

4 files changed, 8 insertions, 15 deletions

diff --git a/src/application.cpp b/src/application.cpp
index 55343030..38ad7477 100644
--- a/src/application.cpp
+++ b/src/application.cpp
@@ -23,6 +23,7 @@
 // ---------------------------------------------------------------------------------------------------------------
 // Ctor and Dtor
 
+// NOLINTNEXTLINE(modernize-avoid-c-arrays)
 Application::Application(int &argc, char *argv[]) : SingleApplication(argc, argv, true)
 {
   // constructor needs to allow secondary instances (allowSecondary = true)
diff --git a/src/application.hpp b/src/application.hpp
index cc38bda9..f435ef8d 100644
--- a/src/application.hpp
+++ b/src/application.hpp
@@ -23,9 +23,9 @@ class RekonqWindow;
 class PluginLoader;
 class RekonqSettings;
 
-typedef QList<QPointer<PluginLoader>> RekonqPluginList;
-typedef QList<QPointer<RekonqWindow>> RekonqWindowList;
-typedef QList<QPointer<RekonqView>> RekonqViewList;
+using RekonqPluginList = QList<QPointer<PluginLoader>>;
+using RekonqWindowList = QList<QPointer<RekonqWindow>>;
+using RekonqViewList = QList<QPointer<RekonqView>>;
 
 // ---------------------------------------------------------------------------------------------------------------
 
@@ -36,7 +36,7 @@ class Application : public SingleApplication {
   Q_OBJECT
 
 public:
-  Application(int &argc, char *argv[]);
+  Application(int &argc, char *argv[]); // NOLINT(modernize-avoid-c-arrays)
   ~Application() override;
 
   REKONQ_TEST_VIRTUAL bool registerPlugin(const QString &path) REKONQ_TEST_PURE;
diff --git a/src/urlbar/urlbar.cpp b/src/urlbar/urlbar.cpp
index f3a34a36..fcd9f7d3 100644
--- a/src/urlbar/urlbar.cpp
+++ b/src/urlbar/urlbar.cpp
@@ -30,8 +30,6 @@ IconButton::IconButton(QWidget *parent) : QToolButton(parent)
   setContextMenuPolicy(Qt::PreventContextMenu);
 }
 
-void IconButton::mouseReleaseEvent(QMouseEvent *event) { emit clicked(event->globalPos()); }
-
 // -----------------------------------------------------------------------------------------------------------
 
 QString guessUrlWithCustomFirstLevel(const QString &str1, const QString &str2)
diff --git a/src/urlbar/urlbar.hpp b/src/urlbar/urlbar.hpp
index da34151e..98e8368d 100644
--- a/src/urlbar/urlbar.hpp
+++ b/src/urlbar/urlbar.hpp
@@ -24,16 +24,10 @@ class IconButton : public QToolButton {
 
 public:
   explicit IconButton(QWidget *parent = nullptr);
-
-signals:
-  void clicked(QPoint);
-
-protected:
-  void mouseReleaseEvent(QMouseEvent *event);
 };
 
 // Definitions
-typedef QList<IconButton *> IconButtonPointerList;
+using IconButtonPointerList = QList<IconButton *>;
 
 class UrlBar : public QLineEdit {
   Q_OBJECT
@@ -91,8 +85,8 @@ public slots:
   // void showSSLInfo(QPoint);
 
 protected:
-  void paintEvent(QPaintEvent *event);
-  void resizeEvent(QResizeEvent *);
+  void paintEvent(QPaintEvent *event) override;
+  void resizeEvent(QResizeEvent *) override;
   /*
       void keyReleaseEvent(QKeyEvent *event);
       void dropEvent(QDropEvent *event);