Enable QT_NO_URL_CAST_FROM_STRING and QT_STRICT_ITERATORS.

* QT_NO_URL_CAST_FROM_STRING makes it clearer where QUrls are created
from QStrings (which incurs a parsing cost).

* QT_STRICT_ITERATORS helps detecting where begin()/end() is used
instead of cbegin()/cend(). KDE developers have verified that
the generated assembly code is identical.

1 files changed, 1 insertions, 1 deletions

diff --git a/lib/room.cpp b/lib/room.cpp
index 1af294a7..a309cd24 100644
--- a/lib/room.cpp
+++ b/lib/room.cpp
@@ -1947,7 +1947,7 @@ void Room::uploadFile(const QString& id, const QUrl& localFilename,
                 });
         connect(job, &BaseJob::success, this, [this, id, localFilename, job] {
             d->fileTransfers[id].status = FileTransferInfo::Completed;
-            emit fileTransferCompleted(id, localFilename, job->contentUri());
+            emit fileTransferCompleted(id, localFilename, QUrl(job->contentUri()));
         });
         connect(job, &BaseJob::failure, this,
                 std::bind(&Private::failedTransfer, d, id, job->errorString()));