cmake: avoid de-duplication of user's CXXFLAGS
[quassel.git] / src / uisupport / action.cpp
index a61339a..d7727a3 100644 (file)
@@ -1,5 +1,5 @@
 /***************************************************************************
- *   Copyright (C) 2005-2018 by the Quassel Project                        *
+ *   Copyright (C) 2005-2022 by the Quassel Project                        *
  *   devel@quassel-irc.org                                                 *
  *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
 
 #include <QApplication>
 
-Action::Action(QObject *parent)
+Action::Action(QObjectparent)
     : QWidgetAction(parent)
 {
-    init();
+    setProperty("isShortcutConfigurable", true);
+    connect(this, &QAction::triggered, this, &Action::slotTriggered);
 }
 
-
-Action::Action(const QString &text, QObject *parent, const QObject *receiver, const char *slot, const QKeySequence &shortcut)
-    : QWidgetAction(parent)
+Action::Action(const QString& text, QObject* parent, const QKeySequence& shortcut)
+    : Action(parent)
 {
-    init();
     setText(text);
     setShortcut(shortcut);
-    if (receiver && slot)
-        connect(this, SIGNAL(triggered()), receiver, slot);
 }
 
-
-Action::Action(const QIcon &icon, const QString &text, QObject *parent, const QObject *receiver, const char *slot, const QKeySequence &shortcut)
-    : QWidgetAction(parent)
+Action::Action(const QIcon& icon, const QString& text, QObject* parent, const QKeySequence& shortcut)
+    : Action(text, parent, shortcut)
 {
-    init();
     setIcon(icon);
-    setText(text);
-    setShortcut(shortcut);
-    if (receiver && slot)
-        connect(this, SIGNAL(triggered()), receiver, slot);
 }
 
-
-void Action::init()
-{
-    connect(this, &QAction::triggered, this, &Action::slotTriggered);
-
-    setProperty("isShortcutConfigurable", true);
-}
-
-
 void Action::slotTriggered()
 {
     emit triggered(QApplication::mouseButtons(), QApplication::keyboardModifiers());
 }
 
-
 bool Action::isShortcutConfigurable() const
 {
     return property("isShortcutConfigurable").toBool();
 }
 
-
 void Action::setShortcutConfigurable(bool b)
 {
     setProperty("isShortcutConfigurable", b);
 }
 
-
 QKeySequence Action::shortcut(ShortcutTypes type) const
 {
     Q_ASSERT(type);
@@ -91,14 +70,12 @@ QKeySequence Action::shortcut(ShortcutTypes type) const
     return shortcuts().isEmpty() ? QKeySequence() : shortcuts().first();
 }
 
-
-void Action::setShortcut(const QShortcut &shortcut, ShortcutTypes type)
+void Action::setShortcut(const QShortcut& shortcut, ShortcutTypes type)
 {
     setShortcut(shortcut.key(), type);
 }
 
-
-void Action::setShortcut(const QKeySequence &key, ShortcutTypes type)
+void Action::setShortcut(const QKeySequence& key, ShortcutTypes type)
 {
     Q_ASSERT(type);