X-Git-Url: https://git.quassel-irc.org/?a=blobdiff_plain;f=src%2Fcore%2Fcoresession.cpp;h=603d291ff7096c7366fc87076379fbe4d74cc00e;hb=3456106b07bfc3000e50df9322ddefd4f872e0fb;hp=0173a9572e2fee6208e8f2fb9f5c1334904cd088;hpb=5cab348de53cb3b994273c06fe69e1f799d247b4;p=quassel.git diff --git a/src/core/coresession.cpp b/src/core/coresession.cpp index 0173a957..603d291f 100644 --- a/src/core/coresession.cpp +++ b/src/core/coresession.cpp @@ -68,6 +68,7 @@ CoreSession::CoreSession(UserId uid, bool restoreState, QObject *parent) _ctcpParser(new CtcpParser(this)), _ircParser(new IrcParser(this)), scriptEngine(new QScriptEngine(this)), + _processMessages(false), _ignoreListManager(this) { SignalProxy *p = signalProxy(); @@ -96,7 +97,6 @@ CoreSession::CoreSession(UserId uid, bool restoreState, QObject *parent) loadSettings(); initScriptEngine(); - connect(eventManager(), SIGNAL(eventQueueEmptied()), SLOT(processMessages())); eventManager()->registerObject(ircParser(), EventManager::NormalPriority); eventManager()->registerObject(sessionEventProcessor(), EventManager::HighPriority); // needs to process events *before* the stringifier! eventManager()->registerObject(ctcpParser(), EventManager::NormalPriority); @@ -223,9 +223,6 @@ void CoreSession::msgFromClient(BufferInfo bufinfo, QString msg) { CoreNetwork *net = network(bufinfo.networkId()); if(net) { net->userInput(bufinfo, msg); - // FIXME as soon as user input is event-based - // until then, user input doesn't trigger a message queue flush! - processMessages(); } else { qWarning() << "Trying to send to unconnected network:" << msg; } @@ -250,6 +247,10 @@ void CoreSession::recvMessageFromServer(NetworkId networkId, Message::Type type, return; _messageQueue << rawMsg; + if(!_processMessages) { + _processMessages = true; + QCoreApplication::postEvent(this, new ProcessMessagesEvent()); + } } void CoreSession::recvStatusMsgFromServer(QString msg) { @@ -270,10 +271,15 @@ QList CoreSession::buffers() const { return Core::requestBuffers(user()); } -void CoreSession::processMessages() { - if(_messageQueue.isEmpty()) +void CoreSession::customEvent(QEvent *event) { + if(event->type() != QEvent::User) return; + processMessages(); + event->accept(); +} + +void CoreSession::processMessages() { if(_messageQueue.count() == 1) { const RawMessage &rawMsg = _messageQueue.first(); bool createBuffer = !(rawMsg.flags & Message::Redirected); @@ -329,6 +335,7 @@ void CoreSession::processMessages() { emit displayMsg(messages[i]); } } + _processMessages = false; _messageQueue.clear(); } @@ -398,7 +405,7 @@ void CoreSession::createIdentity(const Identity &identity, const QVariantMap &ad void CoreSession::createIdentity(const CoreIdentity &identity) { CoreIdentity *coreIdentity = new CoreIdentity(identity, this); _identities[identity.id()] = coreIdentity; - // CoreIdentity has it's own synchronize method since it's "private" sslManager needs to be synced aswell + // CoreIdentity has its own synchronize method since its "private" sslManager needs to be synced as well coreIdentity->synchronize(signalProxy()); connect(coreIdentity, SIGNAL(updated()), this, SLOT(updateIdentityBySender())); emit identityCreated(*coreIdentity); @@ -474,7 +481,10 @@ void CoreSession::removeNetwork(NetworkId id) { return; if(net->connectionState() != Network::Disconnected) { - connect(net, SIGNAL(disconnected(NetworkId)), SLOT(destroyNetwork(NetworkId))); + // make sure we no longer receive data from the tcp buffer + disconnect(net, SIGNAL(displayMsg(NetworkId, Message::Type, BufferInfo::Type, const QString &, const QString &, const QString &, Message::Flags)), this, 0); + disconnect(net, SIGNAL(displayStatusMsg(QString)), this, 0); + connect(net, SIGNAL(disconnected(NetworkId)), this, SLOT(destroyNetwork(NetworkId))); net->disconnectFromIrc(); } else { destroyNetwork(id); @@ -485,6 +495,16 @@ void CoreSession::destroyNetwork(NetworkId id) { QList removedBuffers = Core::requestBufferIdsForNetwork(user(), id); Network *net = _networks.take(id); if(net && Core::removeNetwork(user(), id)) { + // make sure that all unprocessed RawMessages from this network are removed + QList::iterator messageIter = _messageQueue.begin(); + while(messageIter != _messageQueue.end()) { + if(messageIter->networkId == id) { + messageIter = _messageQueue.erase(messageIter); + } else { + messageIter++; + } + } + // remove buffers from syncer foreach(BufferId bufferId, removedBuffers) { _bufferSyncer->removeBuffer(bufferId); }