aacid added a comment.
Then it looks ok to me (well there's no need to move some of the includes to the header, with fordward declarations it would be enough, but who cares i guess :D) Let's see if @tcanabrava has something else to say :) INLINE COMMENTS > cfeck wrote in maindlg.cpp:235 > There is no `addAction(KStandardAction::StandardAction, QString, QObject, > Func)` prototype in `KActionCollection`, so it's OK. oh, someone should fix that :) REPOSITORY R334 KmPlot REVISION DETAIL https://phabricator.kde.org/D17626 To: yurchor, #kde_edu, tcanabrava Cc: cfeck, aacid, tcanabrava, kde-doc-english, kde-edu, skadinna, narvaez, apol
