Room: Slight optimization of makeErrorStr()

The previous version constructed QString from const char* and QByteArray
parts,
only to convert it back to QByteArray; the current version
does the whole thing in QByteArray terms.

1 files changed, 2 insertions, 3 deletions

diff --git a/room.cpp b/room.cpp
index 547b74c4..85953a5c 100644
--- a/room.cpp
+++ b/room.cpp
@@ -406,10 +406,9 @@ void Room::Private::removeMemberFromMap(const QString& username, User* u)
         emit q->memberRenamed(formerNamesakes[0]);
 }
 
-inline QByteArray makeErrorStr(const Event* e, const char* msg)
+inline QByteArray makeErrorStr(const Event* e, QByteArray msg)
 {
-    return QString("%1; event dump follows:\n%2")
-            .arg(msg, QString(e->originalJson())).toUtf8();
+    return msg.append("; event dump follows:\n").append(e->originalJson());
 }
 
 void Room::Private::insertEvent(RoomEvent* e, Timeline::iterator where,