Seems fine to me.  I think the reason we used unix specific libraries originally was we did not plan on supporting plasma on other OSs.  Clearly that's no longer the case so this makes sense.

On Sun, Jun 6, 2010 at 2:08 PM, Alessandro Diaferia <alediaferia@gmail.com> wrote:

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

Ship it!


Can't imagine anything saner than this. Of course last word to Christopher.

- Alessandro


On 2010-06-06 13:15:14, Pino Toscano wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4241/
> -----------------------------------------------------------
>
> (Updated 2010-06-06 13:15:14)
>
>
> Review request for Plasma and Christopher Blauvelt.
>
>
> Summary
> -------
>
> Currently, the soliddevice data engine use either statvfs() or statfs() for determine the free space on a mount point.
> This, other than requiring per-OS code which does not cover all the OS KDE supports (for example Windows), is already wrapped in a nice API in kio, the class KDiskFreeSpaceInfo.
>
> The proposed patch applies this class to the soliddevice data engine, although making it depend on kio.
>
>
> Diffs
> -----
>
>   /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/soliddevice/CMakeLists.txt 1135061
>   /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/soliddevice/soliddeviceengine.cpp 1135061
>
> Diff: http://reviewboard.kde.org/r/4241/diff
>
>
> Testing
> -------
>
> Looks reporting the correct value for mounted mount points, when queried via plasmaengineexplorer.
>
>
> Thanks,
>
> Pino
>
>