[patch] KIconLoader: queryIcons() and queryIconsByContext()

Matt Rogers mattr at kde.org
Sat May 6 15:10:55 BST 2006


On Saturday 06 May 2006 08:51, Luke Sandell wrote:
> Michael Pyne wrote:
> > Good point.  Yes, this should probably be changed for KDE 4 to be public
> > and
> > static.  Better to leave it private for KDE 3.
>
> Why leave for tomorrow what we can do today?  I don't see an binary
> compatibility issues: we're dealing with a nonvirtual member function that
> we're sure is never being referenced outside its own compilation unit
> (KIconLoader has no friends). Unless I'm forgetting something...
>
> -Luke Sandell

changing the visibility of a function breaks BC because it's part of the 
function's signature, IIRC
--
Matt




More information about the kde-core-devel mailing list