Proposal to use QIcon in APIs in KF5.

Tomaz Canabrava tcanabrava at kde.org
Wed Sep 7 06:57:29 BST 2011


On Wed, Sep 7, 2011 at 2:53 AM, Boudewijn Rempt <boud at valdyas.org> wrote:
> On Wednesday 07 September 2011 Sep, John Layt wrote:
>
>> Porting a dozen or so lines from KIcon(foo) to KIconFactory::icon(foo) is
>> nowhere near the effort
>
> Just to make sure that everyone has the right perspective on the size of any porting effort: Calligra alone has over 1200 of KIcon(foo) lines.  KUrl is used about 700 times. There's also about a thousand places where we use kde4_add_library, kde4_add_plugin or kde4_add_unit_test. Simple porting tasks suddenly become a big thing.

and how bad it is to port all functionality from KIcon to QIcon, and
in the KIcon header , a simple

#include <QIcon>
typedef KIcon QIcon

?

then, no porting effort is necessary.

>
> --
> Boudewijn Rempt
> http://www.valdyas.org, http://www.krita.org, http://www.boudewijnrempt.nl
>




More information about the kde-core-devel mailing list