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

List:       kde-devel
Subject:    Re: [PATCH] Use KIconLoader::State in KIconLoader
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2007-12-10 4:43:57
Message-ID: 200712092143.57453.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Sunday 09 December 2007, Bernhard Beschow wrote:
> In the end I think this patch makes the KDE4-API more robust against
> obvious programming mistakes like the mentioned one above. As a side
> effect, it also improves the API documentation. I'd be very happy if you'd
> accept the patch.

the patch makes sense, however it is too late for kde 4.0. the method can be 
marked for removal in kde5 and if feasible an overload can be offered. 


note that the API will not be perfect no matter how long we allow such 
changes, and we passed the line for making such changes some time ago. =/

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

KDE core developer sponsored by Trolltech

[Attachment #5 (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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