Proposal to use QIcon in APIs in KF5.

Olivier Goffart ogoffart at kde.org
Wed Sep 7 10:20:44 BST 2011


On Wednesday 07 September 2011 08:53:25 Boudewijn Rempt 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. 

Try this command:

git grep -O"sed -i 's/KIcon(/QIcon::fromTheme(/g'" KIcon

and I am pretty sure this simple line 'port' over 99%


-- 
Olivier

P.S. sorry for the previous email, I pressed ctrl+enter too early




More information about the kde-core-devel mailing list