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

List:       kde-core-devel
Subject:    Re: BIG CHANGE: loadApplicationIcon -> loadIcon
From:       Stephan Kulow <coolo () kde ! org>
Date:       1999-12-29 20:47:49
[Download RAW message or body]

Kurt Granroth wrote:
> 
> Okay, now that loadApplicationIcon is a kind of the end-all icon
> loader, it makes sense to name it 'loadIcon'.  The current loadIcon
> loads only toolbar icons...
> 
> So, I'm in the processing of doing this:
> 
> 1) Converting all 'loadApplicationIcon' references to 'loadIcon'
> 2) Converting all current 'loadIcon' references to 'BarIcon'
> 
> I will do this for kdelibs, kdebase, kdeutils, kdenetwork, kdeadmin,
> and maybe some others.
> 
> I'll also update KDE2PORTING
> 
> Two questions:
> 
> 1) Should I keep a loadApplicationIcon method with a #warning that
>    it's depreceated?  Or should I just make a hard break?
Prefer compile errors over runtime errors!
> 
> 2) Since we have 'BarIcon', would it make sense to have a 'Icon'?
We had Icon - so applications porting from 1.1 to 2.0 will compile,
but fairly break. So better leave out Icon
> 
> I'll commit the changes tomorrow if all goes as planned.
I wouldn't believe you can do it the way

Greetings, Stephan

-- 
It said Windows 95 or better, so in theory Linux should run it
                                                GeorgeH on /.

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