D7785: Don't make requests if we didn't manage to connect

Aleix Pol Gonzalez noreply at phabricator.kde.org
Tue Sep 12 17:31:55 UTC 2017


apol created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Use QtConcurrent::run instead of QThread. QThread will issue the thread
  immediately and that will overload the operating system giving "too many files
  opened errors". QtConcurrent will do the appropriate balancing for us.

TEST PLAN
  Doesn't crash on me anymore

REPOSITORY
  R134 Discover Software Store

BRANCH
  master

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

AFFECTED FILES
  libdiscover/backends/FlatpakBackend/CMakeLists.txt
  libdiscover/backends/FlatpakBackend/FlatpakBackend.cpp
  libdiscover/backends/FlatpakBackend/FlatpakBackend.h
  libdiscover/backends/FlatpakBackend/FlatpakFetchDataJob.cpp
  libdiscover/backends/FlatpakBackend/FlatpakFetchDataJob.h
  libdiscover/backends/FlatpakBackend/FlatpakFetchUpdatesJob.cpp
  libdiscover/backends/FlatpakBackend/FlatpakFetchUpdatesJob.h
  libdiscover/backends/FlatpakBackend/FlatpakNotifier.cpp
  libdiscover/backends/SnapBackend/SnapBackend.cpp

To: apol, jgrulich
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170912/38197a78/attachment-0001.html>


More information about the Plasma-devel mailing list