Re: [PATCH] speed up stock icon registration (was: Improving Nautilus startup time)

[Alexander Larsson <alexl redhat com>]

On Thu, 2006-03-09 at 16:26 -0500, Matthias Clasen wrote:
> On 3/9/06, Christian Neumair <chris gnome-de org> wrote:
> >
> > The attached patch ensures that we don't hardcode any pixels/icon sizes
> > and don't lookup any stock icons during icon registration, but just add
> > all the stock icon names we need to the factory. Measuring with a GTimer
> > I could reduce the icon registration to rougly 1/4 of the former time,
> > mostly because the gtk_icon_theme_lookup_icon calls are quite expensive.
> > I don't know whether it buys us much to delay the icon size lookups
> > until they're actually needed, but the Nautilus startup feels
> > significantly faster with the patch applied.
> >
> 
> It would be good to understand what this particular nautilus code is trying
> to achieve in the first place. Why does it need to manually register
> icon sources ?

I believe its because we want to use named icons in the menus. And that
doesn't work without registering them first. (If i remember correctly)

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a scarfaced albino ex-con who hides his scarred face behind a mask. She's 
a foxy thirtysomething research scientist trying to make a difference in a 
man's world. They fight crime!