<table><tr><td style="">nicolasfella added a comment.
</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/D17320">View Revision</a></tr></table><br /><div><div><p>Thanks for looking into this!</p>

<p>So now every time we are done with the payload we need to close both the InputStream and the socket? That seems like something that is easily forgotten. Maybe it would make sense to add a closePayload() method to NetworkPacket that closes both so you only need to call one method?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17320">https://phabricator.kde.org/D17320</a></div></div><br /><div><strong>To: </strong>eduisters, KDE Connect<br /><strong>Cc: </strong>nicolasfella, kdeconnect, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>