[patch] KIconLoader: queryIcons() and queryIconsByContext()
michael.pyne at kdemail.net
Sat May 6 06:32:09 BST 2006
On Friday 05 May 2006 11:01, Luke Sandell wrote:
> Michael Pyne wrote:
> > I like the idea, but I don't like the way it's implemented. It's like
> > adding
> > a "no, I'm being an idiot on purpose" flag to a public API.
> Actually removeIconExtension() is currently a private member function. If
> you want to make it public, and I think that it useful enough that it
> should be public, then I can see your point (actually it should be static
> as well).
Good point. Yes, this should probably be changed for KDE 4 to be public and
static. Better to leave it private for KDE 3.
> However, if this is the case, then we'd better not have it print
> warnings to the user (after all, they know their icon has an extension,
> that is why they're calling the function).
> > We would then be able to transition the internal uses of
> > removeIconExtension() in queryIcons() to the
> > internalRemoveIconExtension(), and there would be no code duplication.
> Actually queryIcons() and queryIconsByContext() would still use
> removeIconExtension() (no warnings) whereas the all internal calls would
> use removeIconExtensionInternal() (warnings).
Yes, the fact that I thought removeIconExtension() was public confuses things
but you have figured out what I had in mind.
> If you agree with my points, I will make a patch.
- Michael Pyne
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 191 bytes
Desc: not available
More information about the kde-core-devel