D16475: [SMS App] Make requestMoreMessages asynchronous, blocking, and caching

[Simon Redman]

sredman planned changes to this revision.
sredman added a comment.


  With some time to think about this, I think I can  combine some of Aleix's last comment. Rather than having the worker inherit from QThread, and rather than implicitly expecting the caller to know a QThread is required, the worker could have a QThread which it is in charge of creating, using, and destroying. I think this would make things much nicer.

REPOSITORY
  R224 KDE Connect

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

To: sredman, #kde_connect
Cc: apol, nicolasfella, kdeconnect, shivanshukantprasad, skymoore, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20181107/947aaaca/attachment.html>