[patch] Preload popup menus for the desktop (3.5)
faure at kde.org
Wed Apr 5 19:14:01 BST 2006
On Wednesday 05 April 2006 01:42, Scott Wheeler wrote:
> On Wednesday 05 April 2006 1:19, David Faure wrote:
> > Hmm. I can't see how the plugins would be particularly slower than the rest
> > of the popupmenu-setup code, they do the same kind of thing: creating
> > actions and plugging them. Although arkplugin.cpp does a new KConfig each
> > time which could easily be changed to a kstatic-deleted static KConfig *
> > (but you said kconfig was fast).
> I don't claim to understand it, but for me it's somewhat moot since we allow
> third parties to install stuff there and one poorly written service menu
> entry could make Konq / KDesktop suck without any clear link between the
> plugin and the sucking.
> > Anyway you gave percentages in one mail and milliseconds in the next one
> > so I'm a bit confused with the timing data :)
> Well, the numbers that I'm getting are in ms, when I divide that by the total
> I get a percentage. ;-)
Sure, but without all the numbers it's a bit hard to compare "this code takes 50ms"
and "this other code takes 20%" ;-)
> If you actually want to see what I'm working with
It's the numbers that would interest me more ;)
> +#include <kprofilemethod.h>
Cool - happy to see that my 4-year old hack is still useful :)
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
More information about the kde-core-devel