<table><tr><td style="">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/D16469">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/D16469#inline-89814">View Inline</a><span style="color: #4b4d51; font-weight: bold;">apol</span> wrote in <span style="color: #4b4d51; font-weight: bold;">networkpacket.h:52</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Why do you want this? To be able to access them from QML? DBus?<br />
If it's from QML, you'd be able to make it uncreatable, so this wouldn't be necessary.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The case I hit was being able to use NetworkPacket as the argument to a signal/slot connection across threads. I don't know if this is a useful usecase.</p>

<p style="padding: 0; margin: 8px;">So I guess this is more of a discussion of whether or not we ever think we want to make the daemon multi-threaded. I think the answer to that question is "no". If we think merging this patch won't cause us trouble, we can merge it on the offchance that we do find it useful (for that or something else). On the other hand, I can just abandon this revision and we can dig it up in 100 years when we actually need it 🙂</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/D16469">https://phabricator.kde.org/D16469</a></div></div><br /><div><strong>To: </strong>sredman, KDE Connect<br /><strong>Cc: </strong>albertvaka, 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>