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

List:       kde-devel
Subject:    Re: kpixmapcache vs kiconloader
From:       Stephan Kulow <coolo () alpha ! tat ! physik ! uni-tuebingen ! de>
Date:       1999-05-25 20:08:42
[Download RAW message or body]

David Faure wrote:
> 
> On Tue, May 25, 1999 at 09:28:16PM +0200, Stephan Kulow wrote:
> > Hi!
> >
> > As far as I can see, both do almost the same. Is there a
> > reason for Kpixmapcache or can it be merged into kiconloader?
> 
> I first thought that they should be merged (which means that kiconloader would get
> all the additional methods that exist in kpixmapcache).
> BUT : this would mean that the result of the merge would have
> to be in kio, because it uses the mimetype stuff !
> (e.g. pixmapForURL(...), pixmapForMimeType(...))
> => in fact I see no way of merging.
> 
> We don't want all apps that use the icon loader to be required to link to kio...
> The same problem ever and ever again....
> 
> The only thing that could be removed from kpixmapcache are toolbarPixmap and wallpaperPixmap.

Then I propose to make class kiopixmapcache : public kiconloader

Will look into it.

Greetings, Stephan

-- 
Und sie nannten ihn, wie er selbst unterschrieb -
Den Trojanischen Pferdedieb

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