[PATCH] Fix KActionCategory::add()

Michael Jansen kde at michael-jansen.biz
Sat Nov 1 17:09:53 GMT 2008


On Saturday 01 November 2008 11:21:09 Aurélien Gâteau wrote:
> Hello,
>
> KActionCategory::add() is broken right now, it misses a
> KActionCategory::add(QAction*).
>
> Attached patch fix this. Ok to commit?
>
> Aurélien

Just use 

KAction * addAction(const QString &name, QAction *action);

You HAVE to provide a name. And the actions HAS TO BE a member of the 
associated action collection. Your new method breaks that requirement.

So just tell me your use case.

Mike


-- 
Michael Jansen

http://www.michael-jansen.biz




More information about the kde-core-devel mailing list