[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.


Michael Jansen


More information about the kde-core-devel mailing list