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

List:       kde-devel
Subject:    Re: unnecessary libs ?
From:       aleXXX <alexander.neundorf () gmx ! net>
Date:       2002-08-01 20:37:03
[Download RAW message or body]

On Thursday 01 August 2002 01:41, CP Hennessy wrote:
> Hi,
> 	While looking around the code for kicker I noticed that
> libkonq.so is linked into kickermain.so but I think that it is
> not necessary. When I removed libkonq from the Makefile.am
> (see attched patch), I was able to run kicker, appletproxy and the
> other applets with no problems, and even better the following
>
> > time DISPLAY= appletproxy
>
> seems to show that it saves 0.35 of a second !
> (This is on a 700Mhz machine with 512 MB ).
>
> Maybe there is something I don't understand about linking this
> lib ?
>
> Anyway this led me to look around at other apps and I have a few
> questions :
>
> 1)	Other places where libkonq is linked is
> 	kdesktop.so, keditbookmarks.so, kicker.so

kdesktop shares a lot of code with konqy,  all the file management operations 
you can do on the desktop live in libkonq, also some parts of the icon stuff, 
so this one really depends on libkonq.

> 	but I think that for all 3 of them all they really need is
> 	the KBookmarkManager class ( and related Bookmark classes ).
> 	Linking with libkonq just for these classes is quite
> 	expensive for startup time (see above for appletproxy
> 	startup times) and memory consumed (sorry, no figures at
> 	hand but I have seem 3MB less *per* appletproxy at
> 	work );
>
> 2)	libkio.so also has libkdesu.so linked, but only the class
> 	KDEsuClient seems to be used. Is it worth moving
> 	KDEsuClient into libkio, to save the time for linking with
> 	libkdesu.so ?
>
> Thanks for any answers,
> CP

Nice that somebody looks for these things :-)
In some cases there are indeed unnecessary libs, just watch out carefully. If 
you find something, test it, send a patch, if nobody objects, apply it.
If it was wrong, you won't miss it ;-)

Bye
Alex

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