Add missing Q_REQUIRED_RESULT

src/frame/frame.h

@@ -45,7 +45,7 @@ public:
     * all tabs but the main tab can be removed.
     * Default is @c true
     */
-    bool isRemovable() const;
+    Q_REQUIRED_RESULT bool isRemovable() const;
 
     /**
      * returns the URL of the embedded part

src/frame/framemanager.cpp

@@ -260,7 +260,7 @@ void FrameManager::saveProperties(KConfigGroup &config)
         // No need to save the main frame
         Frame *currentFrame = i.value();
         if (currentFrame && qobject_cast<WebEngineFrame *>(currentFrame)) {
-            QString newPrefix = QLatin1Char('T') + QString::number(i.key());
+            const QString newPrefix = QLatin1Char('T') + QString::number(i.key());
             if (currentFrame->saveConfig(config, newPrefix + QLatin1Char('_'))) {
                 strlst.append(newPrefix);
                 if (currentFrame == m_currentFrame) {

src/frame/mainframe.h

@@ -22,7 +22,7 @@ public:
 
     Q_REQUIRED_RESULT QUrl url() const override;
 
-    bool openUrl(const OpenUrlRequest &) override
+    Q_REQUIRED_RESULT bool openUrl(const OpenUrlRequest &) override
     {
         return false;
     }

src/frame/webengine/webengineframe.h

@@ -21,10 +21,10 @@ public:
     explicit WebEngineFrame(KActionCollection *ac, QWidget *parent = nullptr);
     ~WebEngineFrame() override;
 
-    QUrl url() const override;
-    bool openUrl(const OpenUrlRequest &request) override;
+    Q_REQUIRED_RESULT QUrl url() const override;
+    Q_REQUIRED_RESULT bool openUrl(const OpenUrlRequest &request) override;
     void loadConfig(const KConfigGroup &, const QString &) override;
-    bool saveConfig(KConfigGroup &, const QString &) override;
+    Q_REQUIRED_RESULT bool saveConfig(KConfigGroup &, const QString &) override;
 
     Q_REQUIRED_RESULT qreal zoomFactor() const override;