D7111: Create QMenu on the stack

Denis Kurz noreply at phabricator.kde.org
Fri Aug 4 12:08:21 BST 2017


dkurz added a comment.


  The reasoning is the same as in https://phabricator.kde.org/D7107. Additionally, we can see here how it simplifies memory management and prevents memory leaks: In line 2756, we had to remember to delete menu before we return. We can easily forget that, and now get it for free.

REPOSITORY
  R206 KMail

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

To: dkurz, #kde_pim_kmail
Cc: mlaurent, #kde_pim, dvasin, winterz, vkrause, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170804/6b5f9c5e/attachment.html>


More information about the kde-pim mailing list