D5409: Queue some connections which cannot be autodetected

Aleix Pol Gonzalez noreply at phabricator.kde.org
Wed Apr 12 13:28:52 UTC 2017


apol added inline comments.

INLINE COMMENTS

> KNSBackend.cpp:330
>          m_page = 0;
> -        connect(this, &KNSBackend::receivedResources, stream, &ResultsStream::resourcesFound);
> +        connect(this, &KNSBackend::receivedResources, stream, &ResultsStream::resourcesFound, Qt::QueuedConnection);
>          connect(this, &KNSBackend::searchFinished, stream, &ResultsStream::deleteLater);

Then maybe it's KNS that should be syncing?
i.e. `connect(m_engine, &KNSCore::Engine::signalEntriesLoaded, this, &KNSBackend::receivedEntries);`

Or even have KNSCore emit from the object's thread:
http://doc.qt.io/qt-5/threads-qobject.html#signals-and-slots-across-threads

REPOSITORY
  R134 Discover Software Store

REVISION DETAIL
  https://phabricator.kde.org/D5409

To: leinir, #discover_software_store, apol
Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170412/b913bce8/attachment.html>


More information about the Plasma-devel mailing list