D7529: KIO: fix long-standing memory leak on exit.

Milian Wolff noreply at phabricator.kde.org
Sun Aug 27 20:33:28 UTC 2017


mwolff accepted this revision.
mwolff added a comment.
This revision is now accepted and ready to land.


  lgtm, despite the `delete this`: https://isocpp.org/wiki/faq/freestore-mgmt#delete-this, esp. if you have proper unit tests and ran it through valgrind

REPOSITORY
  R241 KIO

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D7529

To: dfaure, mwolff
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170827/0eb440f6/attachment.html>


More information about the Kde-frameworks-devel mailing list