<table><tr><td style="">dmitrio reclaimed this revision.<br />dmitrio 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/D10663">View Revision</a></tr></table><br /><div><div><p>Well, sorry, I did not expect you waiting for me. I had some new version of this patch, but thought about working on unit tests for it and never finished them. I'll upload now what I have to date, hope it will be useful.</p>

<p>Concerning the <tt style="background: #ebebeb; font-size: 13px;">ERR_DISK_FULL</tt> case which was mentioned earlier, it is easy to add some handling for this case in the new version of patch . However, I should note that at least file ioslave for Unix-like systems also tries to remove partially copied file in case of full disk (see e.g. <a href="https://phabricator.kde.org/source/kio/browse/master/src/ioslaves/file/file_unix.cpp$314" class="remarkup-link" target="_blank" rel="noreferrer">this</a>). Not sure how it would interfere with more high-level treatment in <tt style="background: #ebebeb; font-size: 13px;">FileCopyJob</tt> and whether we should keep or remove that code in case we decide to handle this case in <tt style="background: #ebebeb; font-size: 13px;">FileCopyJob</tt>.</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/D10663">https://phabricator.kde.org/D10663</a></div></div><br /><div><strong>To: </strong>dmitrio, Frameworks, dfaure<br /><strong>Cc: </strong>elvisangelaccio, ngraham, anthonyfieroni, meven, Frameworks, michaelh<br /></div>