<table><tr><td style="">sredman created this revision.<br />sredman added a reviewer: KDE Connect.<br />Herald added a project: KDE Connect.<br />Herald added a subscriber: kdeconnect.<br />sredman requested review of this revision.
</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/D16470">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Data from multiple sources, such as incoming network packets and dbus calls, can currently cause the entire application thread to block if one call needs to wait on another. This diff is the start of moving LanLinkProvider onto its own thread, such that incoming NetworkPackets can be handled independently for different devices and independently of dbus calls. The question at this point is "Do we even want this"?</p>

<p>Current status is that the daemon crashes if the desktop responds to Android's identity broadcast, but if the desktop initiates the connection it works... Until you try to disconnect!<br />
Packet handling is not actually on its own thread yet</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Build app</li>
<li class="remarkup-list-item">Connect devices</li>
<li class="remarkup-list-item">Use as normal (many crashes ensue)</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>BRANCH</strong><div><div>multi-thread-link-provider</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16470">https://phabricator.kde.org/D16470</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>core/backends/lan/lanlinkprovider.cpp<br />
core/backends/linkprovider.cpp<br />
core/backends/linkprovider.h<br />
core/daemon.cpp</div></div></div><br /><div><strong>To: </strong>sredman, KDE Connect<br /><strong>Cc: </strong>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, apol<br /></div>