Review Request: check if enough disk space available before even starting to copy each file

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103412/#review9029
-----------------------------------------------------------



kio/kio/copyjob.cpp
<http://git.reviewboard.kde.org/r/103412/#comment7474>

    Please use KFileSystemType instead of KMountPoint, it will be much faster.
    
    And yeah, given that copying to NFS will be slow anyway, I'm not sure if using KDiskFreeSpaceInfo is really a problem...


- David Faure


On Dec. 16, 2011, 1:28 p.m., Nick Shaforostoff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103412/
> -----------------------------------------------------------
> 
> (Updated Dec. 16, 2011, 1:28 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> this is simple fix for 243160. It gets free space info for the dst partition, then after each successful file copy it decreases an internally kept m_freeSpace value.
> 
> this will help us avoid situations when user copies a 4gb long file onto his disk, then finds out it has not enough space available.
> 
> i hope that i correctly understood kio copy job mechanism and done error reporting right.
> 
> TODO (from what was asked in the bug):
> *checking for single file size limit on vfat.
> *if the total size is larger than free space, warn user beforehand immediately
> (right now it does the copying until it finds that the next file cannot be copied completely)
> both these require new dialogs with user visible strings and are subject to be added after 4.8.
> 
> 
> also as a bonus i changed m_overwriteList to be qset instead of qlist to make lookup operations faster.
> 
> 
> This addresses bug 243160.
>     http://bugs.kde.org/show_bug.cgi?id=243160
> 
> 
> Diffs
> -----
> 
>   kio/kio/copyjob.cpp eff7825 
> 
> Diff: http://git.reviewboard.kde.org/r/103412/diff/diff
> 
> 
> Testing
> -------
> 
> files get copied fine. if i copy a bunch of files including one big file
> (created with dd if=/dev/zero of=file.out bs=1MB count=300), then the copying process stops when it gets to this big file (i have small disk on my virtual machine, only 400 mb free)
> 
> 
> Thanks,
> 
> Nick Shaforostoff
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111217/969f2574/attachment.htm>