KIcon API replacement (Re: patches on kcolors)

David Faure faure at kde.org
Sat Mar 10 12:03:10 UTC 2012


On Saturday 10 March 2012 11:49:21 David Faure wrote:
> Maybe we want to make it a method of 
> KIconLoader/KIconEngine instead?

KIconLoader::loadIcon is already taken (and returns a QPixmap), and 
KIconEngine is an internal class.

So a new suggestion would be:

namespace KDE {
  QIcon loadIcon(const QString& iconName, KIconLoader* iconLoader = 0, const 
QStringList& overlays = QStringList());
}

Doesn't help finding out which header file it would come from, though.
But hopefully, this being in kiconloader.h wouldn't be a big surprise.

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5



More information about the Kde-frameworks-devel mailing list