<table><tr><td style="">sredman marked an inline comment as done.<br />sredman added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16475">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16475#inline-89803">View Inline</a><span style="color: #4b4d51; font-weight: bold;">apol</span> wrote in <span style="color: #4b4d51; font-weight: bold;">requestconversationworker.h:34</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Two things:<br />
Do we still need threads now that we don't have the QEventLoop? I don't think so.<br />
If we did, maybe it would be useful if this class inherited QThread.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">We still need something to solve the same problem as the QEventLoop was trying to solve. That is, we need some way to wait until the remote (Android) device replies with the messages we requested, otherwise we don't have anything to return to the user request and the GUI remains annoyingly empty until they make a second request. If we wait on the main thread, the event loop can't process the incoming packet, so we can never stop waiting</p>

<p style="padding: 0; margin: 8px;">This could be done by inheriting from QThread, but the Qt people are very firm that inheriting from QThread to do a simple task is the wrong way to do it. The way I have done it is the recommended way</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16475">https://phabricator.kde.org/D16475</a></div></div><br /><div><strong>To: </strong>sredman, KDE Connect<br /><strong>Cc: </strong>apol, nicolasfella, kdeconnect, 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<br /></div>