[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: network dataengine in kdereview
From: Kevin Ottens <ervin () kde ! org>
Date: 2008-08-25 22:05:40
Message-ID: 200808260005.40378.ervin () kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
Le Monday 25 August 2008, Aaron J. Seigo a écrit :
> On Monday 25 August 2008, Gerhard Gappmeier wrote:
> > Hi Aaron,
> >
> > I just took a quick look at the code.
> >
> > If noticed that readNumberFromFile() and readStringFromFile() are using
> > fopen to access the file.
>
> the only place that those are used is actually commented out right now;
> they are used for statistics liket tx/rx/etc. that needs to be ported to
> ksysguard or dropped.
I'd say they'd better be dropped for now (I mean they're unused ATM, and
available in the history if we need to bring them back later).
> personally, i'm not sure we need those stats here and that they may be
> better served by either being in another engine or by being separate
> sources in the same engine.
Well, it'd probably make sense to have them in a more "ksysguard like" engine.
In fact if they're removed from this engine it'd probably make sense to also
increase the minimumPollingInterval (twice a second right now, it seems a bit
high to me).
Regards.
--
Kévin 'ervin' Ottens, http://ervin.ipsquad.net
"Ni le maître sans disciple, Ni le disciple sans maître,
Ne font reculer l'ignorance."
["signature.asc" (application/pgp-signature)]
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic