Review Request: proper fix for my inexcusable commit without review

Jaime Torres Amate jtamate at gmail.com
Fri Jun 29 17:22:36 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105364/
-----------------------------------------------------------

(Updated June 29, 2012, 4:22 p.m.)


Review request for kdelibs.


Changes
-------

It is not possible to fix the memory leak in void KAction::setAuthAction(const QString &actionName) the way I tried.
kdelibs will have it until frameworks is ready.
Therefore, replace the delete by just two comments.


Description
-------

the action created locally in void KAction::setAuthAction(const QString &actionName) is deleted.
The others must be handled by the callers of void KAction::setAuthAction(KAuth::Action *action).


Diffs (updated)
-----

  kdeui/actions/kaction.cpp 0f6343c 

Diff: http://git.reviewboard.kde.org/r/105364/diff/


Testing
-------

working with it this afternoon.


Thanks,

Jaime Torres Amate

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120629/dcac076f/attachment.htm>


More information about the kde-core-devel mailing list