[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [PATCH] Add CPU speed detection to KPersonalizer for FBSD
From:       jbarnes () sgi ! com (Jesse Barnes)
Date:       2003-11-15 4:55:21
[Download RAW message or body]

On Fri, Nov 14, 2003 at 09:42:05AM +0100, Stephan Kulow wrote:
> On Thursday 13 November 2003 20:21, Adriaan de Groot wrote:
> > Attached patch is a first stab at adding CPU-speed detection to
> > KPersonalizer for FBSD. For the vast majority of FBSD systems out there
> > the speed will still be detected as 0 [which makes me wonder, why is the
> > CPU speed detected at all, and what is KPersonalizer doing with it?], but
> > in some cases (-CURRENT, ACPI enabled) the right speed is returned.
> > 
> > The patch also corrects an indentation mistake and adds a comment that the
> > CPU speed is expressed in MHz.
> > 
> > OK to commit? Please CC me on replies.
> I'd prefer to remove that whole Mhz handling completly. The MHzs of a computer
> is so useless as measurement how much icon resizing and stuff is wanted. Especially
> since much of it is quite pointless if you only have 32MB of RAM :(

Seems like the real fix would be to time a small but representative
benchmark (e.g. icon resizing and/or svg rendering) and base your
defaults on that.  That's probably too much trouble though, so maybe it
is best to just drop the feature.

Jesse
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]