<table><tr><td style="">dvratil requested changes to this revision.<br />dvratil added a comment.<br />This revision now requires changes to proceed.
</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/D18558">View Revision</a></tr></table><br /><div><div><p>The <tt style="background: #ebebeb; font-size: 13px;">readAll()</tt>s after <tt style="background: #ebebeb; font-size: 13px;">close()</tt> were needed IIRC, because reopening the socket and reading from it caused junk to be read from the buffer after reopening. This may have been caused by me doing something wrong. However, to make sure this does not keep happening, I'd suggest to always recreate the socket in <tt style="background: #ebebeb; font-size: 13px;">doReconnect()</tt> instead of just reconnecting if it already exists.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R165 Akonadi </div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18558">https://phabricator.kde.org/D18558</a></div></div><br /><div><strong>To: </strong>dfaure, dvratil, sengels<br /><strong>Cc: </strong>mlaurent, kde-pim, dvasin, rodsevich, winterz, vkrause, knauss, dvratil<br /></div>