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

List:       kde-commits
Subject:    Re: kdelibs/kdeui
From:       "Aaron J. Seigo" <aseigo () olympusproject ! org>
Date:       2003-04-30 16:53:05
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wednesday 30 April 2003 04:17, Frerich Raabe wrote:
> In any case, I do like it this way better and I can see the marginal
> performance impact caused but since it's my time which was spent on doing
> that work, and since you do not seem to be sufficiently annoyed by it,
> maybe we can leave it as it is?

it bothers me aesthetically, which is why i posted my original question ... 
cornelius and lubos have both echoed my largely unspoken thoughts on the 
matter quite accurately. i appreciate your work, especially since it revolves 
around the generally un-sexy task of core code maintenance...

perhaps we can agree, however, that this is probably not an optimal 
solution... i really hate to undo the work people have put effort into, 
especially in the case of this class where the impacts are quite minimal (if 
unnecessary) and there is reason for what you did ...

perhaps we can, however, settle on a going-forward policy of not doing this to 
other classes.. in fact, i think it may be a very useful discussion whether 
we want to move everything into or out of d pointers as much as possible when 
we next break binary compat... i'm not aware of any KDE policy on that 
matter, and it may be useful to discuss it in advance?

if so, perhaps we could move this discussion to kde-core-devel...

- -- 
Aaron J. Seigo
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE: The 'K' is for 'kick ass'
http://www.kde.org       http://promo.kde.org/3.1/feature_guide.php
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE+r/9x1rcusafx20MRAk+YAJ0e4Hg/c24ZJ5wwzd7+Ne2ECBHIqwCeNP6H
uqFs7fJPx5VC2zAw4fS0Vwo=
=O2wF
-----END PGP SIGNATURE-----
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic