--===============1148977428==
Content-Type: multipart/alternative; boundary=0016362844fa444d63048867f994

--0016362844fa444d63048867f994
Content-Type: text/plain; charset=ISO-8859-1

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
> >
> >
>
>

--0016362844fa444d63048867f994
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Seems fine to me.=A0 I think the reason we used unix specific libraries ori=
ginally was we did not plan on supporting plasma on other OSs.=A0 Clearly t=
hat&#39;s no longer the case so this makes sense.<br><br><div class=3D"gmai=
l_quote">
On Sun, Jun 6, 2010 at 2:08 PM, Alessandro Diaferia <span dir=3D"ltr">&lt;<=
a href=3D"mailto:alediaferia@gmail.com">alediaferia@gmail.com</a>&gt;</span=
> wrote:<br><blockquote class=3D"gmail_quote" style=3D"border-left: 1px sol=
id rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class=3D"im"><br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
</div><a href=3D"http://reviewboard.kde.org/r/4241/#review6000" target=3D"_=
blank">http://reviewboard.kde.org/r/4241/#review6000</a><br>
-----------------------------------------------------------<br>
<br>
Ship it!<br>
<br>
<br>
Can&#39;t imagine anything saner than this. Of course last word to Christop=
her.<br>
<br>
- Alessandro<br>
<div class=3D"im"><br>
<br>
On 2010-06-06 13:15:14, Pino Toscano wrote:<br>
&gt;<br>
&gt; -----------------------------------------------------------<br>
&gt; This is an automatically generated e-mail. To reply, visit:<br>
&gt; <a href=3D"http://reviewboard.kde.org/r/4241/" target=3D"_blank">http:=
//reviewboard.kde.org/r/4241/</a><br>
&gt; -----------------------------------------------------------<br>
&gt;<br>
</div>&gt; (Updated 2010-06-06 13:15:14)<br>
<div><div></div><div class=3D"h5">&gt;<br>
&gt;<br>
&gt; Review request for Plasma and Christopher Blauvelt.<br>
&gt;<br>
&gt;<br>
&gt; Summary<br>
&gt; -------<br>
&gt;<br>
&gt; Currently, the soliddevice data engine use either statvfs() or statfs(=
) for determine the free space on a mount point.<br>
&gt; 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 ki=
o, the class KDiskFreeSpaceInfo.<br>
&gt;<br>
&gt; The proposed patch applies this class to the soliddevice data engine, =
although making it depend on kio.<br>
&gt;<br>
&gt;<br>
&gt; Diffs<br>
&gt; -----<br>
&gt;<br>
&gt; =A0 /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/soliddevic=
e/CMakeLists.txt 1135061<br>
&gt; =A0 /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/soliddevic=
e/soliddeviceengine.cpp 1135061<br>
&gt;<br>
&gt; Diff: <a href=3D"http://reviewboard.kde.org/r/4241/diff" target=3D"_bl=
ank">http://reviewboard.kde.org/r/4241/diff</a><br>
&gt;<br>
&gt;<br>
&gt; Testing<br>
&gt; -------<br>
&gt;<br>
&gt; Looks reporting the correct value for mounted mount points, when queri=
ed via plasmaengineexplorer.<br>
&gt;<br>
&gt;<br>
&gt; Thanks,<br>
&gt;<br>
&gt; Pino<br>
&gt;<br>
&gt;<br>
<br>
</div></div></blockquote></div><br>

--0016362844fa444d63048867f994--

--===============1148977428==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

--===============1148977428==--