<table><tr><td style="">dfaure 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/D22528">View Revision</a></tr></table><br /><div><div><p>Please wait.</p>

<p>This is a partial revert of my earlier commit to this file -- not exactly, but it feels like there is a bit of a mess regarding the question of whether error() was already emitted or not.<br />
I made the assumption that statusServerError meant error was already emitted (by earlier function calls) but it now seems this assumption was wrong.<br />
But then it feels like we're playing ping-pong with this code in a way that will never be satisfactory, if various error cases have or have not already emitted error() at this point.<br />
This needs further research and cleanups.</p>

<p>Well, presumably we're better off with this patch in than out, so actually if you want to commit, I (or someone else...) can do the research either way.</p>

<p>Does anyone know which methods are supposed to call error() and which methods are supposed to just set iError and let the caller do it? Right now it seems to be a bit of a mess...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22528">https://phabricator.kde.org/D22528</a></div></div><br /><div><strong>To: </strong>ZaWertun, sitter, dfaure, cfeck<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns<br /></div>