Convert null strings to empty strings
authorManuel Nickschas <sputnick@quassel-irc.org>
Thu, 30 Sep 2010 07:25:39 +0000 (09:25 +0200)
committerManuel Nickschas <sputnick@quassel-irc.org>
Wed, 13 Oct 2010 23:06:32 +0000 (01:06 +0200)
The storage backend bails out when we hand over null strings, so convert those
to empty strings instead when transforming a MessageEvent into a Message. Null
strings are nicer to use and more efficient, so don't force people to avoid them
in the event handlers.

src/core/coresession.cpp

index a33b942..1f6f324 100644 (file)
@@ -254,8 +254,11 @@ void CoreSession::recvStatusMsgFromServer(QString msg) {
 }
 
 void CoreSession::processMessageEvent(MessageEvent *event) {
-  recvMessageFromServer(event->networkId(), event->msgType(), event->bufferType(), event->target(),
-                        event->text(), event->sender(), event->msgFlags());
+  recvMessageFromServer(event->networkId(), event->msgType(), event->bufferType(),
+                        event->target().isNull()? "" : event->target(),
+                        event->text().isNull()? "" : event->text(),
+                        event->sender().isNull()? "" : event->sender(),
+                        event->msgFlags());
 }
 
 QList<BufferInfo> CoreSession::buffers() const {