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

List:       kde-commits
Subject:    Re: kdenetwork/kbiff
From:       Stephan Kulow <coolo () itm ! mu-luebeck ! de>
Date:       1999-05-26 10:06:17
[Download RAW message or body]

David Faure wrote:
> 
> On Wed, May 26, 1999 at 10:11:00AM +0200, Stephan Kulow wrote:
> > CVS by granroth wrote:
> > > 
> > > kdenetwork/kbiff kbiffmonitor.cpp,1.31,1.32 kbiffmonitor.h,1.17,1.18 \
> > >                 notify.cpp,1.8,1.9 setupdlg.cpp,1.39,1.40 status.cpp,1.2,1.3 \
> > >                 status.h,1.1,1.2
> > > Author: granroth
> > > CVSROOT: /home/kde
> > > Wed May 26 06:05:33 MET DST 1999
> > > Update of /home/kde/kdenetwork/kbiff
> > > In directory zeus:/tmp/cvs-serv22184
> > > 
> > > Modified Files:
> > > kbiffmonitor.cpp kbiffmonitor.h notify.cpp setupdlg.cpp
> > > status.cpp status.h
> > > Log Message:
> > > o 90% of all layouts look decent again.
> > > o Minor tweaks in stack vs heap allocated class members
> > > 
> > > Remaining tweaks needed for 2.0:
> > > o Fix the icon layout
> > > o Figure out what KConfig is doing!  It's reading in all of my string lists as
> > > one big string...
> > > o ICON doesn't find my kbiff.xpm anymore!!!  AUGH!
> > 
> > Because ICON is made for toolbar icons and nothing more!
> > 
> Oops, my fault then. This icon stuff is pretty confusing.
That's why I cleared it and cleaned up the API while I was at it.
To keep something confusing means to bear the code that leads out of
it ;(

The large thing is another interesting example. It has been a hack 
and shouldn't really be in KIconLoader anymore.

> 
> This should be loadApplicationIcon, then ?

Right. Or just use KApplication::getApplicationIcon, which should
exactly
this for you.

Greetings, Stephan

-- 
As long as Linux remains a religion of freeware fanatics,
Microsoft have nothing to worry about.  
                       By Michael Surkan, PC Week Online


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