<table><tr><td style="">dfaure accepted this revision.<br />dfaure added inline comments.<br />This revision is now accepted and ready to land.
</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/D29485">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/D29485#inline-169298">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ahmadsamir</span> wrote in <span style="color: #4b4d51; font-weight: bold;">copyjob.cpp:477</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I meant connection to the remote ulr/host; however e.g Dolphin already reports those connection errors in the status bar when it can't connect, so the comment is redundant now... (I tried to keep the original code/comments in tact, apparently that backfired spectacularly).</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I'm assuming the TODO was about "What if I'm using a NFS mount and the connection breaks at the time of KDiskFreeSpaceInfo, i.e. what should we do about error handling".</p>

<p style="padding: 0; margin: 8px;">But I think the current code -- which ignores errors and moves on, both for local and now for remote files, actually makes most sense. This is after all just a preliminary check. The worst that will happen is that there will indeed not be enough room and the copy will fail. But that's better than not trying at all, possibly due to a bug in one of those two classes, or possibly because of intermittent network failures.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>l-freespace-remote-2 (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29485">https://phabricator.kde.org/D29485</a></div></div><br /><div><strong>To: </strong>ahmadsamir, Frameworks, dfaure, meven, sitter<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns<br /></div>