diff --git a/src/popups/UserMentions.cpp b/src/popups/UserMentions.cpp
index b0223f7fb79cf19d6770df0581d835896320017b..2e70dbd3e999d95399a9bc9e0130f92c816766d9 100644
--- a/src/popups/UserMentions.cpp
+++ b/src/popups/UserMentions.cpp
@@ -74,7 +74,7 @@ UserMentions::initializeMentions(const QMap<QString, mtx::responses::Notificatio
         nhlog::ui()->debug("Initializing " + std::to_string(notifs.size()) + " notifications.");
 
         for (const auto &item : notifs) {
-                for (const auto notif : item.notifications) {
+                for (const auto &notif : item.notifications) {
                         const auto event_id = QString::fromStdString(utils::event_id(notif.event));
 
                         try {
diff --git a/src/timeline/DelegateChooser.cpp b/src/timeline/DelegateChooser.cpp
index 632a2a646eeaa2cd7090bf0753268b0ffdc4a7e3..46ab6c0e68cf0622d0bbe432619b711225e5d4f1 100644
--- a/src/timeline/DelegateChooser.cpp
+++ b/src/timeline/DelegateChooser.cpp
@@ -91,7 +91,7 @@ DelegateChooser::clearChoices(QQmlListProperty<DelegateChoice> *p)
 void
 DelegateChooser::recalcChild()
 {
-        for (const auto choice : choices_) {
+        for (const auto choice : qAsConst(choices_)) {
                 auto choiceValue = choice->roleValue();
                 if (!roleValue_.isValid() || !choiceValue.isValid() || choiceValue == roleValue_) {
                         if (child) {