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

List:       kde-devel
Subject:    Re: [PATCH] powerdevil-remove-cpufreq.patch
From:       Dario Freddi <drf54321 () gmail ! com>
Date:       2010-03-19 13:02:57
Message-ID: 201003191403.03526.drf54321 () gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Friday 19 March 2010 13:56:48 Holger Macht wrote:
> On Fri 19. Mar - 13:39:47, Dario Freddi wrote:
> > (I'm dropping in now that the waters seems quiet)
> > 
> > Holger,
> > 
> > I have in mind a decent refactoring in powerdevil in the near future, due
> > to some changes in Solid::Control that probably would be nice to have. I
> > would appreciate if you could join in and helping out in it.
> 
> Yes, I hope to help as time permits. I just like to see my first
> cleanups commited until going further.

Which reminds me: did you see my comment on your patch on RB?

> 
> > That said, my humble opinion.
> > 
> > This is "show the remaining time in the battery applet" all the way. And
> > I think a similar solution could be taken. So: remove the control from
> > the UI, but leave a "hidden" option in the profiles file letting the
> > advanced user set his governor. What do you guys think? I think this
> > should give out the best of both worlds.
> 
> Just for the record: I'm still against that.
> 
> However, if that's the only way to get the stuff out of the UI, fine
> with me.

It's not really the only way: it's to keep most people happy. After all, you 
would have to edit a text fine inside your .kde/ folder and know what to do. 
So it's almost like it's not present.

> 
> Regards,
>  Holger
> 
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to
> >> unsubscribe <<

-- 
-------------------

Dario Freddi
KDE Developer
GPG Key Signature: 511A9A3B

["signature.asc" (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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

Configure | About | News | Add a list | Sponsored by KoreLogic