<table><tr><td style="">ngraham 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/D16249">View Revision</a></tr></table><br /><div><div><p><tt style="background: #ebebeb; font-size: 13px;">1ul << 32) -1</tt> needs a comment explaining what it means, for people like me with puny brains. :)</p>

<p>There are <em>two</em> checks in this file that result in an <tt style="background: #ebebeb; font-size: 13px;">ERR_DISK_FULL</tt> if there is not enough space. The FAT32 check is now before the second one, but not the first one, so the first one still will still look for adequate space before checking to see whether or not the copy is even possible due to FAT32 file size restrictions.</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/D16249">https://phabricator.kde.org/D16249</a></div></div><br /><div><strong>To: </strong>shubham, ngraham, elvisangelaccio, Frameworks<br /><strong>Cc: </strong>cfeck, bruns, kde-frameworks-devel, michaelh, ngraham<br /></div>