D7107: Move QMenu allocation from heap to stack

[Daniel Vrátil]

dvratil added a comment.


  From my point of view, I think it's a harmless change that makes the code better readable, safer and faster (even though the performance gain here has a negligible impact). I don't see a reason to not merge it.

REPOSITORY
  R206 KMail

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

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