From 732b43cbd6479acb75141d5e83cc51dfdeb33401 Mon Sep 17 00:00:00 2001
From: Laurent Montel <montel@kde.org>
Date: Wed, 12 Jul 2023 13:28:19 +0200
Subject: [PATCH] Use const'ref

---
 src/actionshandler.h          | 2 +-
 src/filetransferpseudojob.cpp | 6 +++---
 src/filetransferpseudojob.h   | 6 +++---
 src/linkpreviewer.cpp         | 2 +-
 src/linkpreviewer.h           | 2 +-
 src/login.h                   | 2 +-
 6 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/actionshandler.h b/src/actionshandler.h
index dd2502b8b..f39ab0a07 100644
--- a/src/actionshandler.h
+++ b/src/actionshandler.h
@@ -47,7 +47,7 @@ public:
 
 Q_SIGNALS:
     void roomChanged();
-    void showEffect(QString effect);
+    void showEffect(const QString &effect);
 
 public Q_SLOTS:
 
diff --git a/src/filetransferpseudojob.cpp b/src/filetransferpseudojob.cpp
index 15ea161ca..b87c357d1 100644
--- a/src/filetransferpseudojob.cpp
+++ b/src/filetransferpseudojob.cpp
@@ -14,7 +14,7 @@ FileTransferPseudoJob::FileTransferPseudoJob(Operation operation, const QString
 {
 }
 
-void FileTransferPseudoJob::fileTransferProgress(QString id, qint64 progress, qint64 total)
+void FileTransferPseudoJob::fileTransferProgress(const QString &id, qint64 progress, qint64 total)
 {
     if (id != m_eventId) {
         return;
@@ -23,7 +23,7 @@ void FileTransferPseudoJob::fileTransferProgress(QString id, qint64 progress, qi
     setTotalAmount(Unit::Bytes, total);
 }
 
-void FileTransferPseudoJob::fileTransferCompleted(QString id, QUrl localFile)
+void FileTransferPseudoJob::fileTransferCompleted(const QString &id, const QUrl &localFile)
 {
     Q_UNUSED(localFile);
     if (id != m_eventId) {
@@ -32,7 +32,7 @@ void FileTransferPseudoJob::fileTransferCompleted(QString id, QUrl localFile)
     emitResult();
 }
 
-void FileTransferPseudoJob::fileTransferFailed(QString id, QString errorMessage)
+void FileTransferPseudoJob::fileTransferFailed(const QString &id, const QString &errorMessage)
 {
     if (id != m_eventId) {
         return;
diff --git a/src/filetransferpseudojob.h b/src/filetransferpseudojob.h
index b747f6b72..ac8a783ec 100644
--- a/src/filetransferpseudojob.h
+++ b/src/filetransferpseudojob.h
@@ -26,17 +26,17 @@ public:
     /**
      * @brief Set the current number of bytes transferred.
      */
-    void fileTransferProgress(QString id, qint64 progress, qint64 total);
+    void fileTransferProgress(const QString &id, qint64 progress, qint64 total);
 
     /**
      * @brief Set the file transfer as complete.
      */
-    void fileTransferCompleted(QString id, QUrl localFile);
+    void fileTransferCompleted(const QString &id, const QUrl &localFile);
 
     /**
      * @brief Set the file transfer as failed.
      */
-    void fileTransferFailed(QString id, QString errorMessage = {});
+    void fileTransferFailed(const QString &id, const QString &errorMessage = {});
 
     /**
      * @brief Start the file transfer.
diff --git a/src/linkpreviewer.cpp b/src/linkpreviewer.cpp
index a990832ce..0dd96849b 100644
--- a/src/linkpreviewer.cpp
+++ b/src/linkpreviewer.cpp
@@ -10,7 +10,7 @@
 
 using namespace Quotient;
 
-LinkPreviewer::LinkPreviewer(QObject *parent, NeoChatRoom *room, QUrl url)
+LinkPreviewer::LinkPreviewer(QObject *parent, NeoChatRoom *room, const QUrl &url)
     : QObject(parent)
     , m_currentRoom(room)
     , m_loaded(false)
diff --git a/src/linkpreviewer.h b/src/linkpreviewer.h
index c545859ec..52c387d66 100644
--- a/src/linkpreviewer.h
+++ b/src/linkpreviewer.h
@@ -45,7 +45,7 @@ class LinkPreviewer : public QObject
     Q_PROPERTY(QUrl imageSource READ imageSource NOTIFY imageSourceChanged)
 
 public:
-    explicit LinkPreviewer(QObject *parent = nullptr, NeoChatRoom *room = nullptr, QUrl url = {});
+    explicit LinkPreviewer(QObject *parent = nullptr, NeoChatRoom *room = nullptr, const QUrl &url = {});
 
     [[nodiscard]] QUrl url() const;
     void setUrl(QUrl);
diff --git a/src/login.h b/src/login.h
index b1139399a..bb85ee1ce 100644
--- a/src/login.h
+++ b/src/login.h
@@ -112,7 +112,7 @@ Q_SIGNALS:
     void loginFlowsChanged();
     void ssoUrlChanged();
     void connected();
-    void errorOccured(QString message);
+    void errorOccured(const QString &message);
     void testingChanged();
     void isLoggingInChanged();
     void isLoggedInChanged();